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

Start caching registry requests and integrate with catalog reader #10

Open
tomzo opened this issue Oct 11, 2018 · 0 comments
Open

Start caching registry requests and integrate with catalog reader #10

tomzo opened this issue Oct 11, 2018 · 0 comments

Comments

@tomzo
Copy link
Member

tomzo commented Oct 11, 2018

In LiGet we have following approach for caching, which can be ported to BaGet:

  1. Every minute we read events from catalog reader of nuget.org.
  2. For any package ID, when an event happens for it, we invalidate cache entries for it.

This allows to cache queries for all versions of packages. BaGet can know all versions of each package, without having to do lookup to nuget.org each time. This allows to serve some queries directly by baget, rather than making extra trip to nuget.org.
In worst case, BaGet does not know about packages which were pushed/unlisted up to 1 minute ago.

This should be done after profiling #9 , we should know if there is actually significant benefit from this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant