[Java] Remove AWS SDK v1 version property and dependency management from parent POM#507
Open
jvdadda wants to merge 1 commit intoawslabs:masterfrom
Open
Conversation
…ent POM Remove the aws.sdk.v1.version property and the aws-java-sdk-kinesis dependency management entry, as all modules have been migrated to v2.
jvdadda
force-pushed
the
remove-sdk-v1-parent-pom-cleanup
branch
from
5fc008b to
146a7f9
Compare
There was a problem hiding this comment.
Pull request overview
This PR completes the removal of AWS SDK v1 dependencies from the project, as part of issue #500, migrating the last remaining SDK v1 references to SDK v2 after the SDK v1 end-of-support on December 31, 2025. The amazon-kinesis-producer (KPL) dependency is intentionally kept since it is a separate library from the AWS Java SDK v1.
Changes:
- Remove the
aws.sdk.v1.versionproperty andaws-java-sdk-kinesisdependency management entry from the parent POM, and removeaws-java-sdk-stsdependencies fromserializer-deserializerandmultilang-schema-registrymodules. - Migrate the
PutRecordGetRecordExamplefrom AWS SDK v1 Kinesis client to SDK v2KinesisClient, updating all request/response types and builder patterns accordingly. - Replace SDK v1
Regions.getCurrentRegion()with SDK v2DefaultAwsRegionProviderChainfor integration test region resolution, and update Javadoc comments fromAWSCredentialsProvidertoAwsCredentialsProvider.
Reviewed changes
Copilot reviewed 1 out of 1 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
pom.xml |
Remove aws.sdk.v1.version property and aws-java-sdk-kinesis dependency management entry |
serializer-deserializer/pom.xml |
Remove aws-java-sdk-sts (v1) dependency |
multilang-schema-registry/pom.xml |
Remove aws-java-sdk-sts (v1) dependency |
examples/pom.xml |
Switch Kinesis dependency from SDK v1 to SDK v2 |
examples/.../PutRecordGetRecordExample.java |
Migrate Kinesis client usage from SDK v1 to SDK v2 with builder patterns |
integration-tests/.../GlueSchemaRegistryConnectionProperties.java |
Replace SDK v1 Regions with SDK v2 DefaultAwsRegionProviderChain |
serializer-deserializer/.../AWSKafkaAvroDeserializer.java |
Update Javadoc from AWSCredentialsProvider to AwsCredentialsProvider |
serializer-deserializer/.../GlueSchemaRegistryKafkaDeserializer.java |
Update Javadoc from AWSCredentialsProvider to AwsCredentialsProvider |
serializer-deserializer/.../AWSKafkaAvroDeserializerTest.java |
Update test comment from AWSCredentialsProvider to AwsCredentialsProvider |
serializer-deserializer/.../GlueSchemaRegistryKafkaDeserializerTest.java |
Update test comment from AWSCredentialsProvider to AwsCredentialsProvider |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review. Take the survey.
emilie-deltil
approved these changes
Mar 9, 2026
jvdadda
changed the title
This was referenced Mar 10, 2026
Closed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Issue #, if available: Closes #500
Description of changes:
Final cleanup to complete the removal of AWS SDK v1 references:
aws.sdk.v1.versionproperty from the parent POMcom.amazonaws:aws-java-sdk-kinesisdependency management entryAll modules have been migrated to AWS SDK v2 in the preceding PRs.
Requires #503, #504, #505, #506, and #508 to be merged first.
This PR chain (#503 → #504 → #505 → #506 → #508 → #507) also addresses #274 (AWS SDK v2 upgrade plans) and supersedes #338 which partially addressed the same goal.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.