+ private int retryCountLimit = 5;
+ @Resource
+ protected Logger logger = Logger.NULL;
+
+ public boolean shouldRetryRequest(HttpCommand command, HttpResponse
response) {
+ if (command.getFailureCount() retryCountLimit) {
+ return false;
+ }
+ if
Closed #285.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/285
Committed to master and 1.7.x.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/285#issuecomment-34800494
+ }
+
+ @Inject(optional = true)
+ @Named(Constants.PROPERTY_MAX_RETRIES)
+ private int retryCountLimit = 5;
+ @Resource
+ protected Logger logger = Logger.NULL;
+
+ public boolean shouldRetryRequest(HttpCommand command, HttpResponse
response) {
+ if
Committed to master and 1.7.x.
Ah, I see my review is already a little late :-( @andrewgaul: most of the
questions are minor, the only one that we may really need to look at, I think,
is about making sure we increment the failure count correctly.
---
Reply to this email directly or view it on
You can merge this Pull Request by running:
git pull https://github.com/maginatics/jclouds atmos-500-1040
Or you can view, comment on it, or merge it online at:
https://github.com/jclouds/jclouds/pull/285
-- Commit Summary --
* JCLOUDS-137: Retry on HTTP 500 AtmosError 1040
-- File
@shrinandj relevant to your interests.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/285#issuecomment-34682671
The turnaround time on this is INCREDIBLE!! :+1:
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/285#issuecomment-34685027
[jclouds ยป jclouds
#827](https://buildhive.cloudbees.com/job/jclouds/job/jclouds/827/) SUCCESS
This pull request looks good
[(what's this?)](https://www.cloudbees.com/what-is-buildhive)
---
Reply to this email directly or view it on GitHub:
[jclouds-java-7-pull-requests
#1056](https://jclouds.ci.cloudbees.com/job/jclouds-java-7-pull-requests/1056/)
UNSTABLE
Looks like there's a problem with this pull request
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/285#issuecomment-34687928
[jclouds-pull-requests
#584](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/584/) SUCCESS
This pull request looks good
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/285#issuecomment-34687898
Spurious test failure, tracked by
[JCLOUDS-429](https://issues.apache.org/jira/browse/JCLOUDS-429).
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/285#issuecomment-34688336
+ private int retryCountLimit = 5;
+ @Resource
+ protected Logger logger = Logger.NULL;
+
+ public boolean shouldRetryRequest(HttpCommand command, HttpResponse
response) {
+ if (command.getFailureCount() retryCountLimit) {
+ return false;
+ }
+ if
13 matches
Mail list logo