-
Notifications
You must be signed in to change notification settings - Fork 4.8k
HIVE-24484: Upgrade Hadoop to 3.3.1 And Tez to 0.10.2 #3279
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
|
My Last run here had 4 errors, of which I think I have fixed 3 more. The one remaining is some XML parsing error, which I think might get auto resolved or may be an after affect. @kgyrtkirk I have sorted the JLine issue here as well, which you told in the previous PR. Can you give a check once |
...java/org/apache/hadoop/hive/ql/security/authorization/StorageBasedAuthorizationProvider.java
Show resolved
Hide resolved
storage-api/src/java/org/apache/hadoop/hive/common/ValidReadTxnList.java
Show resolved
Hide resolved
streaming/src/test/org/apache/hive/streaming/TestStreaming.java
Outdated
Show resolved
Hide resolved
hcatalog/core/src/test/java/org/apache/hive/hcatalog/mapreduce/TestHCatMultiOutputFormat.java
Outdated
Show resolved
Hide resolved
...-unit/src/test/java/org/apache/hadoop/hive/ql/parse/TestReplicationOnHDFSEncryptedZones.java
Show resolved
Hide resolved
...-unit/src/test/java/org/apache/hadoop/hive/ql/parse/TestReplicationOnHDFSEncryptedZones.java
Show resolved
Hide resolved
abstractdog
left a comment
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.
minor note, otherwise LGTM +1
| @@ -1,3 +1,5 @@ | |||
| --! qt:disabled:disabled Tests the output of LS and that changes, Not a functional test, just adds some masking logic | |||
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.
please include hadoop upgrade as the cause in this comment, otherwise looks good to me
abstractdog
left a comment
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.
LGTM +1
|
Merged. Thanx @abstractdog, @kgyrtkirk and @steveloughran for helping with reviews. :-) |
|
also, let me grab the opportunity to thank @belugabehr who put enormous efforts into the hadoop upgrade in the early days! |
…(Ayush Saxena, reviewed by Laszlo Bodor, Zoltan Haindrich and Steve Loughran)
…(Ayush Saxena, reviewed by Laszlo Bodor, Zoltan Haindrich and Steve Loughran)
…(Ayush Saxena, reviewed by Laszlo Bodor, Zoltan Haindrich and Steve Loughran) (cherry picked from commit 8e0a1ec)
…(Ayush Saxena, reviewed by Laszlo Bodor, Zoltan Haindrich and Steve Loughran)
…(Ayush Saxena, reviewed by Laszlo Bodor, Zoltan Haindrich and Steve Loughran) - PARTIAL (cherry picked from commit 8e0a1ec)
…(Ayush Saxena, reviewed by Laszlo Bodor, Zoltan Haindrich and Steve Loughran) - PARTIAL (cherry picked from commit 8e0a1ec)
Upgrade to Hadoop-3.3.1 & Tez to 0.10.2