Skip to content

Build: Bump software.amazon.awssdk:bom from 2.42.41 to 2.44.0#16279

Merged
huaxingao merged 1 commit into
apache:mainfrom
huaxingao:awssdk-bom-2.44.0
May 11, 2026
Merged

Build: Bump software.amazon.awssdk:bom from 2.42.41 to 2.44.0#16279
huaxingao merged 1 commit into
apache:mainfrom
huaxingao:awssdk-bom-2.44.0

Conversation

@huaxingao
Copy link
Copy Markdown
Contributor

Supersedes #16273.

The dependabot PR fails CI because the BOM bump cascades to 44 transitive AWS SDK artifact version changes, but the runtime-deps baseline files weren't regenerated. :iceberg-aws-bundle:checkRuntimeDeps then fails build configuration, which transitively fails build-checks, core-tests, and kafka-connect-tests.

This PR makes the same BOM bump and also regenerates the affected runtime-deps baselines:

  • aws-bundle/runtime-deps.txt — 44 AWS SDK artifacts bumped to 2.44.0.
  • kafka-connect/kafka-connect-runtime/runtime-deps.txt — 39 AWS SDK artifacts bumped to 2.44.0.

Verified locally:

$ ./gradlew :iceberg-aws-bundle:checkRuntimeDeps \
            :iceberg-kafka-connect:iceberg-kafka-connect-runtime:checkRuntimeDeps

BUILD SUCCESSFUL

Same pattern as #16206.

Made with Cursor

Co-authored-by: Cursor <cursoragent@cursor.com>
Copy link
Copy Markdown
Contributor

@singhpk234 singhpk234 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, but this is a pretty big bump in SDK pov 2 major release, do we know what all precisely went in and are we confident all works

NOTE: There is no integ test against actual AWS for this

@huaxingao
Copy link
Copy Markdown
Contributor Author

Full changelog is here: https://github.com/aws/aws-sdk-java-v2/blob/master/CHANGELOG.md

I don't have an AWS setup to run the integration tests.

@singhpk234
Copy link
Copy Markdown
Contributor

singhpk234 commented May 11, 2026

I checked 2.44.x & 2.43.x https://github.com/aws/aws-sdk-java-v2/releases/tag/2.43.0 changelogs too it doesn't look concerning to me, yes the integ test would have given confidence, but that true that requires setup lets for now go ahead and merge this if something breaks (since we are not releasing this in 1.11, since branch cut happened), we would be good

@huaxingao huaxingao merged commit f86a672 into apache:main May 11, 2026
37 checks passed
@huaxingao
Copy link
Copy Markdown
Contributor Author

Thanks @singhpk234 for the review!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants