steveloughran commented on PR #7130: URL: https://github.com/apache/hadoop/pull/7130#issuecomment-2443966471
thanks for the update. There's a jersey 2 pr right now, #7019, which is also patching the CI build in some way. I think that one needs to go in first, with jackson support coming in later. It would probably be a good think to be a bit ore proactive, but openrewrite seems to have restricted itself to the remapping of imports other than the rest of the complexity. Before worrying about that work then, adding and moving to our own Nullable/NotNullable etc would be good, and can be done component by component. We did it for @VisibleForTesting a while back, as well as other Guava stuff (preconditions, rate limiting,...). essentially: anything with a public api MUST NOT use third party libraries as part of the API. -- 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: common-issues-unsubscr...@hadoop.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
