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

service/kinesis: Add support for retrying service specific API errors #2751

Merged
merged 3 commits into from
Aug 14, 2019

Conversation

jasdel
Copy link
Contributor

@jasdel jasdel commented Aug 13, 2019

Adds support for retrying the Kinesis API error, LimitExceededException.

Fix #1376

Adds support for retrying the Kinesis API error, "LimitExceededException".

Fix aws#1376
@jasdel jasdel requested a review from skotambkar August 13, 2019 16:48
@jasdel jasdel self-assigned this Aug 13, 2019
@jasdel jasdel added the needs-review This issue or pull request needs review from a core team member. label Aug 13, 2019
@jasdel jasdel merged commit 0d2fb42 into aws:master Aug 14, 2019
@jasdel jasdel deleted the feat/10_KinesisRetries branch August 14, 2019 20:58
@aws-sdk-go-automation aws-sdk-go-automation mentioned this pull request Aug 15, 2019
bflad added a commit to hashicorp/terraform-provider-aws that referenced this pull request Aug 30, 2019
The AWS Go SDK now automatically retries on this error.

References:

- aws/aws-sdk-go#1376
- #1085
- aws/aws-sdk-go#2751
- #9770

Output from acceptance testing:

```
--- PASS: TestAccAWSKinesisStream_createMultipleConcurrentStreams (205.83s)
```
skotambkar added a commit to aws/aws-sdk-go-v2 that referenced this pull request Sep 17, 2019
Ports v1 SDK's retry utilities to follow a consistent code path. aws.IsErrorRetryable is the primary entry point to determine if a request is retryable. Also ports support for service/kinesis to retry service specific API errors . Corrects sdk's behavior by not retrying errors with status code 501.

Related to aws/aws-sdk-go#2744, aws/aws-sdk-go#2774, aws/aws-sdk-go#2751 and aws/aws-sdk-go#1826

Fixes #372; Fixes #145
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-review This issue or pull request needs review from a core team member.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Kinesis LimitExceededException errors are not retried
2 participants