Skip to content

Conversation

@amanraj2520
Copy link
Contributor

JIRA link : https://issues.apache.org/jira/browse/TEZ-4483. This PR addresses critical vulnerabilities [CVE-2017-20165, CVE-2017-20165]

@amanraj2520
Copy link
Contributor Author

@abstractdog @rbalamohan Can you please approve and merge this.

@tez-yetus
Copy link

🎊 +1 overall

Vote Subsystem Runtime Comment
+0 🆗 reexec 22m 57s Docker mode activated.
_ Prechecks _
+1 💚 dupname 0m 0s No case conflicting files found.
+1 💚 @author 0m 0s The patch does not contain any @author tags.
_ master Compile Tests _
_ Patch Compile Tests _
+1 💚 whitespace 0m 0s The patch has no whitespace issues.
_ Other Tests _
+1 💚 asflicense 0m 53s The patch does not generate ASF License warnings.
24m 30s
Subsystem Report/Notes
Docker ClientAPI=1.42 ServerAPI=1.42 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-277/1/artifact/out/Dockerfile
GITHUB PR #277
Optional Tests dupname asflicense
uname Linux 649f469b6907 4.15.0-206-generic #217-Ubuntu SMP Fri Feb 3 19:10:13 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality personality/tez.sh
git revision master / 25a9536
Max. process+thread count 51 (vs. ulimit of 5500)
modules C: tez-ui U: tez-ui
Console output https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-277/1/console
versions git=2.34.1 maven=3.6.3
Powered by Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@amanraj2520
Copy link
Contributor Author

@abstractdog @rbalamohan Can you please review this

@amanraj2520
Copy link
Contributor Author

Can some OSS committer please review this?

cc @abstractdog @rbalamohan

@abstractdog abstractdog self-requested a review April 3, 2023 09:45
@abstractdog
Copy link
Contributor

tests passed, this LGTM

@amanraj2520
Copy link
Contributor Author

@abstractdog Can we please merge this

@abstractdog
Copy link
Contributor

this turned out to be conflicting due to TEZ-4484, can you please rebase @amanraj2520?

@amanraj2520
Copy link
Contributor Author

Sure @abstractdog will do

@amanraj2520
Copy link
Contributor Author

@abstractdog I have resolved the merge conflicts. Kindly merge this if it looks good.

@tez-yetus
Copy link

🎊 +1 overall

Vote Subsystem Runtime Comment
+0 🆗 reexec 0m 19s Docker mode activated.
_ Prechecks _
+1 💚 dupname 0m 0s No case conflicting files found.
+1 💚 @author 0m 0s The patch does not contain any @author tags.
_ master Compile Tests _
-0 ⚠️ patch 0m 27s Used diff version of patch file. Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.
_ Patch Compile Tests _
+1 💚 whitespace 0m 0s The patch has no whitespace issues.
_ Other Tests _
+1 💚 asflicense 0m 53s The patch does not generate ASF License warnings.
1m 52s
Subsystem Report/Notes
Docker ClientAPI=1.42 ServerAPI=1.42 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-277/2/artifact/out/Dockerfile
GITHUB PR #277
Optional Tests dupname asflicense
uname Linux ace3304fc562 4.15.0-206-generic #217-Ubuntu SMP Fri Feb 3 19:10:13 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality personality/tez.sh
git revision master / 6ca35c1
Max. process+thread count 51 (vs. ulimit of 5500)
modules C: tez-ui U: tez-ui
Console output https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-277/2/console
versions git=2.34.1 maven=3.6.3
Powered by Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@abstractdog abstractdog self-requested a review April 5, 2023 09:16
@abstractdog abstractdog merged commit 9a729cd into apache:master Apr 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants