Skip to content

TEZ-4626: Remove usage of deprecated classes of commons-cli - #411

Merged
abstractdog merged 1 commit into
apache:masterfrom
SourabhBadhya:TEZ-4626
May 5, 2025
Merged

TEZ-4626: Remove usage of deprecated classes of commons-cli#411
abstractdog merged 1 commit into
apache:masterfrom
SourabhBadhya:TEZ-4626

Conversation

@SourabhBadhya

Copy link
Copy Markdown
Contributor

TEZ-4626: Remove usage of deprecated classes of commons-cli

@tez-yetus

This comment was marked as outdated.

@tez-yetus

This comment was marked as outdated.

@tez-yetus

This comment was marked as outdated.

@tez-yetus

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec0m 34sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+0 🆗spotbugs0m 1sspotbugs executables are not available.
+0 🆗detsecrets0m 1sdetect-secrets was not available.
+1 💚@author0m 0sThe patch does not contain any @author tags.
-1 ❌test4tests0m 0sThe patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
_ master Compile Tests _
+0 🆗mvndep2m 12sMaven dependency ordering for branch
+1 💚mvninstall10m 18smaster passed
+1 💚compile2m 21smaster passed with JDK Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu122.04
+1 💚compile2m 12smaster passed with JDK Private Build-1.8.0_442-8u442-b06us1-0ubuntu122.04-b06
+1 💚checkstyle1m 59smaster passed
+1 💚javadoc1m 37smaster passed with JDK Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu122.04
+1 💚javadoc1m 28smaster passed with JDK Private Build-1.8.0_442-8u442-b06us1-0ubuntu122.04-b06
_ Patch Compile Tests _
+0 🆗mvndep0m 10sMaven dependency ordering for patch
+1 💚mvninstall1m 48sthe patch passed
+1 💚codespell0m 33sNo new issues.
+1 💚compile1m 30sthe patch passed with JDK Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu122.04
+1 💚javac1m 30sthe patch passed
+1 💚compile1m 19sthe patch passed with JDK Private Build-1.8.0_442-8u442-b06us1-0ubuntu122.04-b06
+1 💚javac1m 19sthe patch passed
+1 💚blanks0m 0sThe patch has no blanks issues.
+1 💚checkstyle0m 48sthe patch passed
+1 💚javadoc0m 37sthe patch passed with JDK Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu122.04
+1 💚javadoc0m 36sthe patch passed with JDK Private Build-1.8.0_442-8u442-b06us1-0ubuntu122.04-b06
_ Other Tests _
+1 💚unit4m 57stez-dag in the patch passed.
+1 💚unit42m 25stez-tests in the patch passed.
+1 💚unit2m 24stez-history-parser in the patch passed.
+1 💚unit2m 42sjob-analyzer in the patch passed.
+1 💚asflicense0m 45sThe patch does not generate ASF License warnings.
84m 46s
SubsystemReport/Notes
DockerClientAPI=1.49 ServerAPI=1.49 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-411/4/artifact/out/Dockerfile
GITHUB PR#411
Optional Testsdupname asflicense javac javadoc unit spotbugs checkstyle codespell detsecrets compile
unameLinux 10bebf5c767a 5.15.0-136-generic #147-Ubuntu SMP Sat Mar 15 15:53:30 UTC 2025 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personality/home/jenkins/jenkins-home/workspace/tez-multibranch_PR-411/src/.yetus/personality.sh
git revisionmaster / 289aba4
Default JavaPrivate Build-1.8.0_442-8u442-b06us1-0ubuntu122.04-b06
Multi-JDK versions/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu122.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_442-8u442-b06us1-0ubuntu122.04-b06
Test Resultshttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-411/4/testReport/
Max. process+thread count1116 (vs. ulimit of 5500)
modulesC: tez-dag tez-tests tez-plugins/tez-history-parser tez-tools/analyzers/job-analyzer U: .
Console outputhttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-411/4/console
versionsgit=2.34.1 maven=3.6.3 codespell=2.0.0
Powered byApache Yetus 0.15.1 https://yetus.apache.org

This message was automatically generated.

@SourabhBadhya

Copy link
Copy Markdown
ContributorAuthor

@ayushtkn@abstractdog Please review this PR.

@ayushtknayushtkn left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@abstractdog
abstractdog merged commit 9f990d9 into apache:masterMay 5, 2025
Sign up for freeto 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

@SourabhBadhya@tez-yetus@ayushtkn@abstractdog