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

Caching and conditional requests #53

Open
schwern opened this issue Oct 29, 2014 · 1 comment
Open

Caching and conditional requests #53

schwern opened this issue Oct 29, 2014 · 1 comment

Comments

@schwern
Copy link
Contributor

schwern commented Oct 29, 2014

Github supports the awkwardly named Conditional Requests to check if a resource has been modified. This allows one to cache API responses and cheaply check if there has been a change.

I plan on adding cache support to Net::GitHub::V3, on by default, with a simple Least Recently Used cache limited by number of queries stored. I plan on adding it at the query level so every query is cached. This allows Gitpan to make API requests as needed to check for changes rather than try to implement its own caching, make a bunch of redundant API calls, or just hope there's been no change.

What do you think?

@fayland
Copy link
Owner

fayland commented Oct 30, 2014

@schwern please go ahead.

let me know if you want to be a co-maintainer.

Thanks.

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

2 participants