Skip to content

Conversation

@NihalJain
Copy link
Contributor

No description provided.

@Apache-HBase
Copy link

💔 -1 overall

Vote Subsystem Runtime Comment
+0 🆗 reexec 2m 12s 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.
-0 ⚠️ test4tests 0m 0s The 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 _
+1 💚 mvninstall 1m 0s master passed
+1 💚 compile 0m 21s master passed
+1 💚 javadoc 0m 8s master passed
_ Patch Compile Tests _
+1 💚 mvninstall 0m 43s the patch passed
+1 💚 compile 0m 18s the patch passed
+1 💚 javac 0m 18s the patch passed
+1 💚 whitespace 0m 0s The patch has no whitespace issues.
-1 ❌ xml 0m 0s The patch has 1 ill-formed XML file(s).
+1 💚 javadoc 0m 7s the patch passed
_ Other Tests _
+1 💚 unit 0m 46s root in the patch passed.
+1 💚 asflicense 0m 6s The patch does not generate ASF License warnings.
5m 49s
Reason Tests
XML Parsing Error(s):
pom.xml
Subsystem Report/Notes
Docker ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-Thirdparty-PreCommit/job/PR-143/1/artifact/yetus-precommit-check/output/Dockerfile
GITHUB PR #143
Optional Tests dupname asflicense javac javadoc unit xml compile
uname Linux 10eddb5b3050 5.4.0-1103-aws #111~18.04.1-Ubuntu SMP Tue May 23 20:04:10 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
git revision master / fd76b1e
Default Java Eclipse Adoptium-17.0.11+9
xml https://ci-hbase.apache.org/job/HBase-Thirdparty-PreCommit/job/PR-143/1/artifact/yetus-precommit-check/output/xml.txt
Test Results https://ci-hbase.apache.org/job/HBase-Thirdparty-PreCommit/job/PR-143/1/testReport/
Max. process+thread count 404 (vs. ulimit of 1000)
modules C: . U: .
Console output https://ci-hbase.apache.org/job/HBase-Thirdparty-PreCommit/job/PR-143/1/console
versions git=2.34.1 maven=/home/jenkins/jenkins-home/workspace/Base-Thirdparty-PreCommit_PR-143/yetus-precommit-check/src/dev-support/toolchains-jenkins.xml
Powered by Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@NihalJain
Copy link
Contributor Author

Ok so just xml is failing and javac and other are not a problem. CC: @Apache9

@NihalJain
Copy link
Contributor Author

Ok this is fixed with https://issues.apache.org/jira/browse/YETUS-1105

@Apache9
Copy link
Contributor

Apache9 commented Jul 25, 2025

Ok this is fixed with https://issues.apache.org/jira/browse/YETUS-1105

Yes, upgrading to 0.14.1 can fix the problem. Yetus uses xmllint in newer version.

@NihalJain
Copy link
Contributor Author

Ok this is fixed with https://issues.apache.org/jira/browse/YETUS-1105

Yes, upgrading to 0.14.1 can fix the problem. Yetus uses xmllint in newer version.

Will try to sync with hbase: apache/hbase@17ce7c3

@NihalJain
Copy link
Contributor Author

Ok this is fixed with https://issues.apache.org/jira/browse/YETUS-1105

Yes, upgrading to 0.14.1 can fix the problem. Yetus uses xmllint in newer version.

Will try to sync with hbase: apache/hbase@17ce7c3

Raised #144, lets see how it goes!

@NihalJain
Copy link
Contributor Author

Patch is ready at #144, closing this test PR!

@NihalJain NihalJain closed this Jul 25, 2025
@NihalJain NihalJain reopened this Jul 28, 2025
@NihalJain
Copy link
Contributor Author

Let me retry this patch after fix #144 144

@Apache-HBase

This comment was marked as outdated.

@Apache-HBase

This comment was marked as outdated.

@NihalJain NihalJain reopened this Jul 28, 2025
@Apache-HBase
Copy link

💔 -1 overall

Vote Subsystem Runtime Logfile Comment
+0 🆗 reexec 1m 41s Docker mode activated.
_ Prechecks _
+1 💚 dupname 0m 0s No case conflicting files found.
+0 🆗 codespell 0m 1s codespell was not available.
+0 🆗 detsecrets 0m 1s detect-secrets was not available.
+1 💚 @author 0m 0s The patch does not contain any @author tags.
-0 ⚠️ test4tests 0m 0s The 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 _
+1 💚 mvninstall 0m 55s master passed
+1 💚 compile 0m 17s master passed
+1 💚 javadoc 0m 7s master passed
_ Patch Compile Tests _
+1 💚 mvninstall 0m 41s the patch passed
+1 💚 compile 0m 16s the patch passed
+1 💚 javac 0m 16s the patch passed
-1 ❌ blanks 0m 0s /blanks-eol.txt The patch has 1 line(s) that end in blanks. Use git apply --whitespace=fix <<patch_file>>. Refer https://git-scm.com/docs/git-apply
+1 💚 xmllint 0m 0s No new issues.
+1 💚 javadoc 0m 6s the patch passed
_ Other Tests _
+1 💚 unit 0m 46s root in the patch passed.
+1 💚 asflicense 0m 6s The patch does not generate ASF License warnings.
5m 4s
Subsystem Report/Notes
Docker ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-Thirdparty-PreCommit/job/PR-143/3/artifact/yetus-precommit-check/output/Dockerfile
GITHUB PR #143
Optional Tests dupname asflicense javac javadoc unit codespell detsecrets xmllint compile
uname Linux 7f9a3cd37ca5 5.4.0-1103-aws #111~18.04.1-Ubuntu SMP Tue May 23 20:04:10 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality /home/jenkins/jenkins-home/workspace/Base-Thirdparty-PreCommit_PR-143/yetus-precommit-check/src/.yetus/personality.sh
git revision master / 266bdea
Default Java Eclipse Adoptium-17.0.11+9
Test Results https://ci-hbase.apache.org/job/HBase-Thirdparty-PreCommit/job/PR-143/3/testReport/
Max. process+thread count 418 (vs. ulimit of 2000)
modules C: . U: .
Console output https://ci-hbase.apache.org/job/HBase-Thirdparty-PreCommit/job/PR-143/3/console
versions git=2.34.1 maven=called xmllint=20913
Powered by Apache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

@NihalJain
Copy link
Contributor Author

Build passes now, ignore blank error as that is due to dummy change!

@NihalJain
Copy link
Contributor Author

Testing complete, closing now!

@NihalJain NihalJain closed this Jul 31, 2025
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