ramackri commented on code in PR #689:
URL: https://github.com/apache/atlas/pull/689#discussion_r3526949064
##########
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:
Atlas REST still runs on Jersey 1.19 + JAX-RS 1.1 (jsr311-api) via
[AtlasJsonProvider](vscode-file://vscode-app/Applications/Cursor.app/Contents/Resources/app/out/vs/code/electron-sandbox/workbench/webapp/src/main/java/org/apache/atlas/web/util/AtlasJsonProvider.java),
which extends JacksonJaxbJsonProvider.
Jackson JAX-RS dropped JAX-RS 1.x support starting at 2.13. The last line
that works with Jersey 1.x is 2.12.x (2.12.7 is what PR #689 pins).
PR #689 already tried jaxrs at 2.16.2 without the pin — REST failed (GET
/api/atlas/admin/version did not return 200). Pinning jaxrs at 2.12.7 fixed it.
To use jaxrs 2.16.2
We need a Jersey 2.x upgrade ([PR
#640](https://github.com/apache/atlas/pull/640)), not just a version bump in
webapp/pom.xml.
Recommendation for PR #689: Keep the 2.12.7 pin and core/databind is already
at 2.16.2; jaxrs cannot follow until Jersey is upgraded.
--
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]
