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

Migrate jaxb bind dependency to jakarta #17370

Merged
merged 4 commits into from
Oct 27, 2024

Conversation

suraj-goel
Copy link
Contributor

@suraj-goel suraj-goel commented Oct 17, 2024

Fixes #17340

Description

Testing

  • Local Build
  • Local Deployment
  • Deployed on Druid Cluster

This PR has:

  • been self-reviewed.
  • added documentation for new or modified features or behaviors.
  • a release note entry in the PR description.
  • added Javadocs for most classes and all non-trivial methods. Linked related entities via Javadoc links.
  • added or updated version, license, or notice information in licenses.yaml
  • added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
  • added unit tests or modified existing tests to cover new code paths, ensuring the threshold for code coverage is met.
  • added integration tests.
  • been tested in a test Druid cluster.

Release Note

  • Dependency Migration: Migrated XML binding from javax.xml.bind to jakarta.xml.bind to ensure compatibility with the latest standards and ongoing support.
  • Version Update: Minor version incremented to maintain backward compatibility.
  • End-of-Life (EOL) Dependency: Deprecated javax.xml.bind dependency removed; refer to the JAXB Specification (GitHub link) for more details.

@suraj-goel suraj-goel marked this pull request as ready for review October 17, 2024 08:08
Copy link
Contributor

@abhishekrb19 abhishekrb19 left a comment

Choose a reason for hiding this comment

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

Thanks for updating the archived dependency. Could you add a release note for this dependency update so it gets included in the next release?

@suraj-goel
Copy link
Contributor Author

Thanks for updating the archived dependency. Could you add a release note for this dependency update so it gets included in the next release?

Added a release note

@abhishekrb19 abhishekrb19 merged commit 7306d28 into apache:master Oct 27, 2024
91 checks passed
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.

Replace EOL javaee/jaxb-api with jarkartaee/jaxb-api
2 participants