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

aws/request: Do not treat LimitExceededException as throttle error #1276

Merged
merged 1 commit into from
May 16, 2017

Conversation

jasdel
Copy link
Contributor

@jasdel jasdel commented May 16, 2017

The LimitExceededException error code should not be treated as a
throttling condition. This error code implies that resources are
exhausted or too many tasks are being attempted at once. The SDK does
not have the information it needs to distinguish between the resources
exhausted and too many tasks attempted at once.

This logic is best implemented by user applications as they will have
the most context about what is being attempted.

To add custom retry with this error code a custom implementation of the
request.Retryer interface should be used.

Fix #1271

The LimitExceededException error code should not be treated as a
throttling condition. This error code implies that resources are
exhausted or too many tasks are being attempted at once. The SDK does
not have the information it needs to distinguish between the resources
exhausted and too many tasks attempted at once.

This logic is best implemented by user applications as they will have
the most context about what is being attempted.

To add custom retry with this error code a custom implementation of the
"request.Retryer" interface should be used.

Fix aws#1271
@jasdel jasdel requested a review from xibz May 16, 2017 18:12
@jasdel jasdel self-assigned this May 16, 2017
@jasdel jasdel merged commit fed27f4 into aws:master May 16, 2017
@jasdel jasdel deleted the fix/DynamoDBLimitExceeded branch May 16, 2017 23:29
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.

2 participants