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

too many queries to metadata url? #97

Closed
aspyker opened this issue Apr 1, 2014 · 2 comments
Closed

too many queries to metadata url? #97

aspyker opened this issue Apr 1, 2014 · 2 comments
Assignees
Labels

Comments

@aspyker
Copy link
Contributor

aspyker commented Apr 1, 2014

https://github.com/Netflix/eureka/blob/master/eureka-client/src/main/java/com/netflix/appinfo/AmazonInfo.java#L118

autoBuild seems to have an algorithm that retries awsMetaDataRetries in case of error. I expected to see a break from the while (numOfRetries-- > 0) upon success (where it does result.metadata.put(key.getName(), value);). Instead in a version of this we copied and changed, we see it just try every query awsMetaDataRetries times.

We could submit a patch for this when we submit our similar class. However, I wanted to double check that I understood the code right and there is really a issue here.

@NiteshKant
Copy link
Contributor

@aspyker it indeed is a bug. There should be a break after line 136. I will fix this.

@NiteshKant NiteshKant self-assigned this Apr 2, 2014
@NiteshKant NiteshKant added the bug label Apr 15, 2014
NiteshKant pushed a commit to NiteshKant/eureka that referenced this issue Apr 15, 2014
1) Implementing clean shutdown of client. Moved away from Timer to ScheduledExecutorService as it gives better control of shutdown by interrupting the running thread, which is what we want.
 2)
NiteshKant added a commit that referenced this issue Apr 15, 2014
@NiteshKant
Copy link
Contributor

Available in release 1.1.128

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants