-
Notifications
You must be signed in to change notification settings - Fork 12
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
/stream/items/ids limit does not support n > 1000 #20
Comments
Hey @noinnion, thanks for reporting. What do you mean by 2-way sync feature? And what if user has more than 1000 (all or even unread) items? |
in gReader we use /stream/items/ids to synchronize unread items from server to client. i.e. items read on TheOldReader server will be synced back to gReader. If the limit is only 1000 items that are outside these 1000 items will no be recognize. gReader uses incremental synchronization so that only new items are downloaded. Hope you can increase the limit to 10000 as before. |
Ok, I see. So, if user has more than 10000 unread items gReader won't sync them all? (Assuming the limit is 10000) |
10000 should be enough. Everything larger than 10000 could be ignored. |
I'll try to figure how we can fix this asap. In the meantime, is there any reason why gReader doesn't use continuation for any cases when not all data is fetched? I'm pretty sure there are people with 10000 or more items. |
we use continuation to get more than 1k items. but more than 10k is too much for a reader. (/stream/items/ids does not support continuation) We also use nt/ot to optimize synchronization. if ppl have more than 10k items in their inbox that means they are not using it for a while. All these items outside these 10k are not relevant anymore. Also using more than 10000 would slow down the app in term of performance. |
@krasnoukhov I just see that there is a continuation in the response. Thanks |
Ok, limit is 10000 for ids now, please see related note in readme. |
great, thank you for the quick fix |
@krasnoukhov It seems that /stream/items/ids limit does not support more than 1000. It would destroy 2-way sync feature.
The text was updated successfully, but these errors were encountered: