Opened 10 years ago

Closed 10 years ago

#2106 closed enhancement (fixed)

[PATCH] Download MOTD concurrently

Reported by: xeen Owned by: framm
Priority: trivial Milestone:
Component: unspecified Version:
Keywords: Cc:


The attached patch will make use of ExecutorService to download the content parts concurrently. For me with a ping of ~100ms this yields about 15% time savings, but may be better for high-latency connections (from 24938ms to 21234ms for 20 MOTD downloads).

The patch should make it possible to move the point where the MOTD data is required to the end of the loading queue. Making it asynchronous would also work, both make you wait. Async has the advantage though to display an UI even without MOTD… hmm, what's up with the patch from Nov 17. that was posted to the Mailing List ("Make start-up feel faster by asynchronously loading the MOTD")

See it online

Attachments (1)

MakeMOTDconcurrent.patch (6.1 KB) - added by xeen 10 years ago.

Download all attachments as: .zip

Change History (2)

Changed 10 years ago by xeen

Attachment: MakeMOTDconcurrent.patch added

comment:1 Changed 10 years ago by anonymous

Resolution: fixed
Status: newclosed

Both applied in r1350.

I would say patch has been overlooked. Better is attaching them as Trac tickets :-)

Modify Ticket

Change Properties
Set your email in Preferences
as closed The owner will remain framm.
as The resolution will be set.
The resolution will be deleted.

Add Comment

E-mail address and name can be saved in the Preferences.

Note: See TracTickets for help on using tickets.