ramackri commented on code in PR #689:
URL: https://github.com/apache/atlas/pull/689#discussion_r3527313178
##########
webapp/pom.xml:
##########
@@ -58,6 +58,18 @@
<version>${jackson.version}</version>
</dependency>
+ <dependency>
+ <groupId>com.fasterxml.jackson.jaxrs</groupId>
+ <artifactId>jackson-jaxrs-base</artifactId>
+ <version>2.12.7</version>
Review Comment:
Thanks for the suggestion — agree that one Jackson version everywhere is the
cleaner long-term goal.
**What is already unified in this PR:** `jackson-core`, `jackson-databind`,
and `jackson-annotations` are all at **2.16.2** via the root `jackson.version`
/ `jackson.databind.version` properties. Manual Docker testing passed with that
stack and Jersey 1.19.
**Why the split exists:** The only exception is `jackson-jaxrs-base` /
`jackson-jaxrs-json-provider`, pinned at **2.12.7** in `webapp/pom.xml`.
Jackson JAX-RS 2.13+ dropped JAX-RS 1.x support; with Jersey 1.19, pulling
jaxrs **2.16.2** into the WAR broke REST in our testing (`GET
/api/atlas/admin/version` did not return 200). Pinning jaxrs at 2.12.7 +
excluding the 2.16.2 transitives from `atlas-intg` fixed it.
**Follow-up:** Full unification (including jaxrs at 2.16.2) belongs with a
Jersey 2.x migration — see [PR #640](https://github.com/apache/atlas/pull/640)
/ ATLAS-5190 — rather than expanding the Kafka/KRaft scope here.
Added a brief comment in `webapp/pom.xml` pointing to #640 for future
readers.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
