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

Implement retryable() method in exceptions. #3200

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

marcogrcr
Copy link

@marcogrcr marcogrcr commented May 18, 2022

Implement the retryable() method in the AwsServiceException and
SdkServiceException classes.

Motivation and Context

The SDK internally has a way to determine whether an exception is
retryable based on the status code and the error code. This change uses
this existing logic to implement the retryable() method of the
exception classes so that it no longer returns a static false value.

Modifications

Implement the retryable() method in the AwsServiceException and
SdkServiceException classes.

Testing

Successful build of ./mvnw clean install -pl :aws-core -am in:

  • Amazon Linux 2: Linux 5.4.181-109.354.amzn2int.x86_64 #1 SMP Wed Mar 2 18:58:57 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux
  • Java: openjdk version "1.8.0_332" | OpenJDK Runtime Environment (build 1.8.0_332-b08) | OpenJDK 64-Bit Server VM (build 25.332-b08, mixed mode)

Screenshots (if appropriate)

N/A

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)

Checklist

  • I have read the CONTRIBUTING document
  • Local run of mvn install succeeds
  • My code follows the code style of this project
  • My change requires a change to the Javadoc documentation
  • I have updated the Javadoc documentation accordingly
  • I have added tests to cover my changes
  • All new and existing tests passed
  • I have added a changelog entry. Adding a new entry must be accomplished by running the scripts/new-change script and following the instructions. Commit the new file created by the script in .changes/next-release with your changes.
  • My change is to implement 1.11 parity feature and I have updated LaunchChangelog

License

  • I confirm that this pull request can be released under the Apache 2 license

@marcogrcr marcogrcr requested a review from a team as a code owner May 18, 2022 04:54
WHAT?

Implement the `retryable()` method in the `AwsServiceException` and
`SdkServiceException` classes.

WHY?

The SDK internally has a way to determine whether an exception is
retryable based on the status code and the error code. This change uses
this existing logic to implement the `retryable()` method of the
exception classes so that it no longer returns a static `false` value.

HOW?

Successful build of `./mvnw clean install -pl :aws-core -am`
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 13 Code Smells

100.0% 100.0% Coverage
0.0% 0.0% Duplication

aws-sdk-java-automation added a commit that referenced this pull request Aug 26, 2024
…1c3c74fc3

Pull request: release <- staging/fba0090d-d7bc-4a5e-a976-eed1c3c74fc3
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.

1 participant