~reesmichael1/roman#11: 
Feeds should be downloaded in parallel at start

Status
RESOLVED IMPLEMENTED
Submitter
~reesmichael1
Assigned to
No-one
Submitted
11 months ago
Updated
4 months ago
Labels
quality-of-life

~reesmichael1 11 months ago

As of f3c533be, the feed contents are downloaded in parallel. However, actually parsing the feeds turned out to be a larger bottleneck than simply downloading them. The parsing cannot be parallelized yet due to the external dependency on html2text. I'm going to prioritize nim-html2text now so that this dependency can be removed and this issue can be closed.

~reesmichael1 REPORTED IMPLEMENTED 4 months ago

Closed in 1cbed27. It turns out that we had to tell the compiler that calling traverse on the TextContext was GC-safe (presumably because of the forward declaration? I'm not entirely sure), but once that was done, it was pretty easy to actually implement this.

It turns out that it's really only a small performance benefit, but it'll still be nice to have for users with lots of feed subscriptions.

Register here or Log in to comment, or comment via email.