Skip to content

Sleep and retry if raise API rate limit exceeded #173

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

Merged
merged 2 commits into from
Oct 31, 2017
Merged

Conversation

nalabjp
Copy link
Member

@nalabjp nalabjp commented Oct 31, 2017

DoorkeeperのAPIは300 reqs / 5 minutesというレートリミットが設定されており、
3年分弱のイベント履歴の集計を行うとレートリミットに引っかかる可能性がある。
そのため、60秒スリープしてからリトライするようにした。
レートリミットに引っかかることが複数回連続で発生する場合があるが、
概ね3回程度のリトライで継続可能な模様。

@nalabjp nalabjp merged commit 6f23a82 into master Oct 31, 2017
@nalabjp nalabjp deleted the api-rate-limit branch October 31, 2017 18:02
@yasulab yasulab added the 統計情報 Tracking event record function via APIs: https://coderdojo.jp/stats label Jan 18, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
統計情報 Tracking event record function via APIs: https://coderdojo.jp/stats
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants