Skip to content
This repository was archived by the owner on May 13, 2025. It is now read-only.

Using RateLimiter from Guava to avoid triggering GitHub abuse detection mechanism - #90

Merged
cescoffier merged 1 commit into
github:masterfrom
netceler:master
May 24, 2015
Merged

Using RateLimiter from Guava to avoid triggering GitHub abuse detection mechanism#90
cescoffier merged 1 commit into
github:masterfrom
netceler:master

Conversation

@bguerin

Copy link
Copy Markdown
Contributor

Fix for #17, #28 and #88

cescoffier added a commit that referenced this pull request May 24, 2015
Using RateLimiter from Guava to avoid triggering GitHub abuse detection mechanism
@cescoffier
cescoffier merged commit 4354db6 into github:masterMay 24, 2015
@cescoffier

Copy link
Copy Markdown
Contributor

Thanks.

@kevinsawicki

Copy link
Copy Markdown
Contributor

I just deployed a 0.12.0 release with this fix in it, thanks 👍

@bguerin

Copy link
Copy Markdown
ContributorAuthor

Great, thanks !

@mjremijan

Copy link
Copy Markdown

Small tip, use --debug when running Maven. I thought I ran into the #101 problem but it just took a while to generate the large number of blobs. Running with --debug showed the progress

@alessandro-molari

Copy link
Copy Markdown

I'm facing the same problem (using version 0.12).

To create 460 blobs it takes like 1 hour (not joking).

@jakeswenson

jakeswenson commented Jul 12, 2016

Copy link
Copy Markdown

Yes sadly this PR didn't take GHE in to account at all, and there is no way to override this setting... even though there is no way i can exceed my GHE rate limit....

@bguerin

Copy link
Copy Markdown
ContributorAuthor

@jakeswenson GHE ?

@jakeswenson

Copy link
Copy Markdown

@bguerin GHE = Github Enterprise
Basically a corporate github solution

@bguerin

Copy link
Copy Markdown
ContributorAuthor

@jakeswenson ok thanks. How does GHE works ? Higher limit, no limit at all ?

Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants

@bguerin@cescoffier@kevinsawicki@mjremijan@alessandro-molari@jakeswenson@ShangJunFengg@gulian-github