-
Notifications
You must be signed in to change notification settings - Fork 8.9k
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
HADOOP-18033. [WIP] Remove jsr311-api dependency #4460
base: trunk
Are you sure you want to change the base?
Conversation
Build kind of died only. Guess the test failures screwed the build in some way, We already know something will break, just wanted to know what all break and how big is the impact.... |
Exactly, I wish at least we could know all the failures (even though we know we are stuck with this dependency) |
Looks like jersey-core:1.19 depends on jsr311-api and jsr311-api and rs-api cannot be on the classpath together because they clash. |
@pjfanning Cool, if I understood correct, you mean: javax.ws.rs-api can be removed without Jackson downgrade. |
I have #4547 - let's see if that passes CI build |
javax.ws.rs-api was introduced only to support Jackson 2.13 upgrade. Without rs-api, (on Jackson 2.13) majority of Yarn and ATS tests fail as jackson 2.13 does have dependency on JAX-RS 2 based rs-api (PR #3749) |
@virajjasani I think FasterXML/jackson-jaxrs-providers#134 which appeared in jackson-jaxrs for v2.13.0 to be the reason rs-api was added to hadoop - so in #4547, I am looking to downgrade to jackson 2.12.7 |
Exactly, without downgrading Jackson, it's not possible to remove javax.ws.rs-api. |
Description of PR
More discussion on HADOOP-18033