Skip to content

Migrate jaxb bind dependency to jakarta#17370

Merged
abhishekrb19 merged 4 commits intoapache:masterfrom
confluentinc:jaxb-migration-apache
Oct 27, 2024
Merged

Migrate jaxb bind dependency to jakarta#17370
abhishekrb19 merged 4 commits intoapache:masterfrom
confluentinc:jaxb-migration-apache

Conversation

@suraj-goel
Copy link
Copy Markdown
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
Copy Markdown
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?

Comment thread licenses.yaml
@suraj-goel
Copy link
Copy Markdown
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
@adarshsanjeev adarshsanjeev added this to the 32.0.0 milestone Jan 16, 2025
bshrinidhi pushed a commit to acceldata-io/druid that referenced this pull request Apr 9, 2026
- Migrated from javax.xml.bind 2.3.1  to jakarta.xml.bind 2.3.3.
- Minor version is modified to avoid any breaking changes.
bshrinidhi pushed a commit to acceldata-io/druid that referenced this pull request Apr 21, 2026
- Migrated from javax.xml.bind 2.3.1  to jakarta.xml.bind 2.3.3.
- Minor version is modified to avoid any breaking changes.
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

3 participants