Skip to content

Conversation

@vrozov
Copy link
Member

@vrozov vrozov commented Oct 23, 2025

What changes were proposed in this pull request?

Add explicit dependency on javax.xml.bind:jaxb-api to the test scope of Kinesis ASL.

Why are the changes needed?

JDK 9 and above removed javax.xml.bind package from JDK, so when test runs on JDK 9 and above it fails.

Does this PR introduce any user-facing change?

No

How was this patch tested?

with JDK 17
ENABLE_KINESIS_TESTS=1 build/mvn test -Pkinesis-asl -pl connector/kinesis-asl
and
ENABLE_KINESIS_TESTS=1 build/sbt -Pkinesis-asl

Was this patch authored or co-authored using generative AI tooling?

No

@github-actions github-actions bot added the BUILD label Oct 23, 2025
Copy link
Member

@dongjoon-hyun dongjoon-hyun left a comment

Choose a reason for hiding this comment

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

Hi, @vrozov . Currently, this PR has two mixed themes unlike the PR title and description.

Add explicit dependency on javax.xml.bind:jaxb-api to the test scope of Kinesis ASL.

JDK 9 and above removed javax.xml.bind package from JDK, so when test runs on JDK 9 and above it fails.

In order to help your PR get merged, I'd like to recommend to split this PR into two.

  1. One is only chasing javax.xml.bind:jaxb-api (the scope of PR title and description)
  2. The other is refactoring the code via the following.
private lazy val streamExistsWaiter = kinesisClient.waiters().streamExists()

Also, cc @sarutak , too

@vrozov
Copy link
Member Author

vrozov commented Oct 23, 2025

@dongjoon-hyun @sarutak The primary focus of the PR is to fix NoClassDefFoundError. The problem is that without the second change test still fails as Kinesis throws ResourceNotFoundException even after describe stream returns ACTIVE. Please see #52630 (comment). I can split this PR into 2, but one will continue to fail till the second one is merged.

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants