Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Reply rss is too demanding and time-sensetive #268

Closed
umputun opened this issue Jan 27, 2019 · 1 comment
Closed

Reply rss is too demanding and time-sensetive #268

umputun opened this issue Jan 27, 2019 · 1 comment
Assignees
Milestone

Comments

@umputun
Copy link
Owner

umputun commented Jan 27, 2019

Implementation of #125 and #145 works as designed however practically it makes very "dynamic" rss feed. Such feed provides new items only for some relatively short window (currently 24h). Practically it means we have to have some rss reader/client running at least once a day to pick up all the updates. I'm going to change the logic and keep replies in the feed limited by the number of items only.

Another problem - in order to retrieve replies we pull last comments (up to 1000) and this limit can be way too low for deep history or active sites. Unfortunately, the store doesn't provide any better and more effective way to discover chains/parents. I'll see what can be done here, probably this info can be collected as a part of notification logic, but still not very clear how to deal with all of this.

@umputun
Copy link
Owner Author

umputun commented Jan 27, 2019

I have increased limit and window and it should do for the regular activity, but still problematic for heavy sites.

@umputun umputun added this to the v1.3 milestone Apr 15, 2019
@umputun umputun closed this as completed Apr 15, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant