-
Notifications
You must be signed in to change notification settings - Fork 3.4k
HBASE-27148 Move minimum hadoop 3 support version to 3.2.3 #4561
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
Conversation
|
Also removed all the org.codehaus.jackson dependencies, as it is the already EOL and has CVEs, and hadoop should not actually rely on it although it will be pulled in as transitive dependencies |
|
🎊 +1 overall
This message was automatically generated. |
|
💔 -1 overall
This message was automatically generated. |
|
💔 -1 overall
This message was automatically generated. |
Seems to be a bug in HDFS? |
|
🎊 +1 overall
This message was automatically generated. |
|
💔 -1 overall
This message was automatically generated. |
|
💔 -1 overall
This message was automatically generated. |
|
🎊 +1 overall
This message was automatically generated. |
|
💔 -1 overall
This message was automatically generated. |
|
💔 -1 overall
This message was automatically generated. |
|
The pre commit result is a bit strange, as posted above... Let me try hadoop 3.1.x to see whether we still have the same problem... |
|
💔 -1 overall
This message was automatically generated. |
|
🎊 +1 overall
This message was automatically generated. |
|
🎊 +1 overall
This message was automatically generated. |
|
💔 -1 overall
This message was automatically generated. |
|
💔 -1 overall
This message was automatically generated. |
|
🎊 +1 overall
This message was automatically generated. |
|
💔 -1 overall
This message was automatically generated. |
|
💔 -1 overall
This message was automatically generated. |
|
Seems TestMetaFixer is always failing, no matter what is the hadoop version. Let me try locally again. I tried once in the past and there is no problem... |
|
OK, I could reproduce the problem locally through command line |
|
🎊 +1 overall
This message was automatically generated. |
|
💔 -1 overall
This message was automatically generated. |
|
💔 -1 overall
This message was automatically generated. |
|
OK, good. Seems worked. Let me fix the NPE, and also apply it to deprecated HBaseTestingUtility. |
|
🎊 +1 overall
This message was automatically generated. |
|
💔 -1 overall
This message was automatically generated. |
|
💔 -1 overall
This message was automatically generated. |
|
OK, the problem is that avro still depends on jackson 1.x, where I exclude them all because of the vulnerability warnings... Let me see if updating the avro version can solve the problem... |
|
🎊 +1 overall
This message was automatically generated. |
|
💔 -1 overall
This message was automatically generated. |
|
I think avro 1.10 and up will get you clear of them |
|
🎊 +1 overall
This message was automatically generated. |
|
🎊 +1 overall
This message was automatically generated. |
|
🎊 +1 overall
This message was automatically generated. |
|
💔 -1 overall
This message was automatically generated. |
|
🎊 +1 overall
This message was automatically generated. |
|
Seems upgrade to avro 1.11.0 can solve the last problem. The recent two UT failures are not related. Kicked another pre commit run to see if the result is still good. |
|
🎊 +1 overall
This message was automatically generated. |
|
💔 -1 overall
This message was automatically generated. |
Thank you. Upgrading to 1.11.0 can solve the problem :) |
|
I think the PR is ready to land. Let's fix the security warnings from github! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1
Signed-off-by: Xin Sun <[email protected]> (cherry picked from commit 41972cb)
Signed-off-by: Xin Sun <[email protected]> (cherry picked from commit 41972cb)
Signed-off-by: Xin Sun <[email protected]> (cherry picked from commit 41972cb)
Signed-off-by: Xin Sun <[email protected]> (cherry picked from commit 41972cb)
Signed-off-by: Xin Sun <[email protected]> (cherry picked from commit 41972cb)
…4599) Signed-off-by: Xin Sun <[email protected]> (cherry picked from commit 41972cb)
…4599) Signed-off-by: Xin Sun <[email protected]> (cherry picked from commit 41972cb)
No description provided.