Skip to content

Conversation

@amanraj2520
Copy link
Contributor

@amanraj2520 amanraj2520 changed the title [TEZ-4423][CVE-2021-44906] Upgrade minimist version from 0.0.8 to 1.2.6 TEZ-4423[CVE-2021-44906] Upgrade minimist version from 0.0.8 to 1.2.6 Jun 15, 2022
@tez-yetus
Copy link

🎊 +1 overall

Vote Subsystem Runtime Comment
+0 🆗 reexec 1m 2s 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 59s The patch does not generate ASF License warnings.
2m 43s
Subsystem Report/Notes
Docker ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-220/1/artifact/out/Dockerfile
GITHUB PR #220
Optional Tests dupname asflicense
uname Linux 1adba3aacb13 4.15.0-175-generic #184-Ubuntu SMP Thu Mar 24 17:48:36 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality personality/tez.sh
git revision master / 3da8438
Max. process+thread count 46 (vs. ulimit of 5500)
modules C: tez-ui U: tez-ui
Console output https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-220/1/console
versions git=2.25.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 June 15, 2022 11:55
Copy link
Contributor

@abstractdog abstractdog left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

phantomjs tests passed locally, LGTM +1

@amanraj2520
Copy link
Contributor Author

@guptanikhil007 Was working with me on this. Nikhil can you approve? Then we can merge it.

@amanraj2520
Copy link
Contributor Author

@abstractdog We are good to merge these. Can you please do the same.

@abstractdog abstractdog merged commit c6eb047 into apache:master Jun 15, 2022
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.

4 participants