You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
After we fixed postingOffset sampling which was broken (we were keeping in memory all of them, instead of 1/32 as we intended), some of the users see extreme improvement:
This means that the about of postings is extreme in some setups, so what we keep in memory in terms of posting offsets really matters. This means we might need some caching for posting offsets as well. While we store them on disk, still disk lookup might be slower than in memory. I doubt there would be a benefit to storing this in remote cache though.
Hello 👋 Looks like there was no activity on this issue for last 30 days. Do you mind updating us on the status? Is this still reproducible or needed? If yes, just comment on this PR or push a commit. Thanks! 🤗
If there will be no activity for next week, this issue will be closed (we can always reopen an issue if we need!). Alternatively, use remind command if you wish to be reminded at some point in future.
After we fixed postingOffset sampling which was broken (we were keeping in memory all of them, instead of 1/32 as we intended), some of the users see extreme improvement:
This means that the about of postings is extreme in some setups, so what we keep in memory in terms of posting offsets really matters. This means we might need some caching for posting offsets as well. While we store them on disk, still disk lookup might be slower than in memory. I doubt there would be a benefit to storing this in remote cache though.
cc @d-ulyanov @mkabischev
The text was updated successfully, but these errors were encountered: