Better arn parsing

[RanVaknin]

Motivation and Context

Addresses #6108 .

Currently the SDK uses exceptions as control flow for arn parsing. This is an anti pattern in Java and can cause performance overhead.

Modifications

This PR adds another arn parsing method tryFromString(arn) that would return an Optional of Arn when the arn is valid, and an empty when it is invalid - this avoids the hot path of throwing exceptions as control flow.

Testing

Unit tests:

• Valid ARNs
• Invalid ARNs
• Exception throwing ARNs (valid ARNs missing a partition or service)

[davidh44]

Looks good, just need a a changelog

[zoewangg]

Don't we need to update account ID code path to use this new method?

[RanVaknin]

Don't we need to update account ID code path to use this new method?

Separate internal Arn class used in endpoints logic already returns Optional
https://github.com/aws/aws-sdk-java-v2/blob/master/codegen/src/main/resources/software/amazon/awssdk/codegen/rules/Arn.java.resource#L26

[sonarqubecloud]

Quality Gate failed

Failed conditions
0.0% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

[github-actions]

This pull request has been closed and the conversation has been locked. Comments on closed PRs are hard for our team to see. If you need more assistance, please open a new issue that references this one.