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

GitHub response caching #203

Merged
merged 4 commits into from
Jan 20, 2016
Merged

GitHub response caching #203

merged 4 commits into from
Jan 20, 2016

Conversation

czechboy0
Copy link
Member

Added a smart URL response cache, which sends ETag in a header from previously received responses, thus allowing GitHub to return 304 Not Modified. This has a huge advantage for instances running multiple syncers, because cached responses don't count towards the GitHub rate limit, making it much easier to run more syncers and not getting closer to the limit. Now we'll only get a real response once something has changed on the server.

Closes #62.

…reviously received responses, thus allowing GitHub to return 304 Not Modified. This has a huge advantage for instances running multiple syncers, because cached responses don't count towards the GitHub rate limit, making it much easier to run more syncers and not getting closer to the limit. Now we'll only get a real response once something has changed on the server.
…ws for Buildasaur to handle many more syncers with more GitHub data. Now, Buildasaur caches all responses in memory and uses conditional requests to find out what has changed and only downloads that. Not only does this reduce the total bandwidth by more than 95%, it also doesn't count towards to GitHub rate limit, making Buildasaur much more scalable.
@czechboy0
Copy link
Member Author

The interesting work is actually done here: buildasaurs/BuildaUtils@019461a

@buildasaur
Copy link
Collaborator

Result of Integration 1

Duration: 4 minutes and 59 seconds
Result: Perfect build! All 34 tests passed. 👍
Test Coverage: 23%

czechboy0 added a commit that referenced this pull request Jan 20, 2016
@czechboy0 czechboy0 merged commit 2fe0c45 into master Jan 20, 2016
@czechboy0 czechboy0 deleted the hd/github_response_caching branch January 20, 2016 13:38
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

Successfully merging this pull request may close these issues.

Request/response caching
2 participants