Skip to content

HBASE-29860 Fix NullPointerException on Maven site build #7694

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.

Sign up for GitHub

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

Jump to bottom
Open
PDavid wants to merge 1 commit into
base: branch-2.6
Choose a base branch
from
Open

HBASE-29860 Fix NullPointerException on Maven site build #7694

PDavid wants to merge 1 commit into from
+2 −2

Conversation

@PDavid
Copy link
Contributor

@PDavid PDavid commented Jan 29, 2026

This occurs when the parser encounters a section in an .xdoc file that it cannot properly index, such as sections without a name attribute.

@PDavid
HBASE-29860 Fix NullPointerException on Maven site build
7918c59 
This occurs when the parser encounters a section in an .xdoc file that it cannot properly index, such as sections without a name attribute.
@PDavid PDavid self-assigned this Jan 29, 2026
@PDavid PDavid added the bug label Jan 29, 2026
@PDavid PDavid mentioned this pull request Jan 29, 2026
Closed
@Apache-HBase
Copy link

Apache-HBase commented Jan 29, 2026

🎊 +1 overall

Vote Subsystem Runtime Logfile Comment
+0 🆗 reexec 0m 14s Docker mode activated.
-0 ⚠️ yetus 0m 5s Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --author-ignore-list --blanks-eol-ignore-file --blanks-tabs-ignore-file --quick-hadoopcheck
_ Prechecks _
_ branch-2.6 Compile Tests _
_ Patch Compile Tests _
_ Other Tests _
1m 18s
Subsystem Report/Notes
Docker ClientAPI=1.53 ServerAPI=1.53 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7694/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile
GITHUB PR #7694
Optional Tests
uname Linux f17a071fff3d 6.14.0-1018-aws #18~24.04.1-Ubuntu SMP Mon Nov 24 19:46:27 UTC 2025 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality dev-support/hbase-personality.sh
git revision branch-2.6 / 7918c59
Max. process+thread count 40 (vs. ulimit of 30000)
modules C: . U: .
Console output https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7694/1/console
versions git=2.34.1 maven=3.9.8
Powered by Apache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase
Copy link

Apache-HBase commented Jan 29, 2026

🎊 +1 overall

Vote Subsystem Runtime Logfile Comment
+0 🆗 reexec 0m 14s Docker mode activated.
-0 ⚠️ yetus 0m 5s Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --author-ignore-list --blanks-eol-ignore-file --blanks-tabs-ignore-file --quick-hadoopcheck
_ Prechecks _
_ branch-2.6 Compile Tests _
_ Patch Compile Tests _
_ Other Tests _
1m 17s
Subsystem Report/Notes
Docker ClientAPI=1.53 ServerAPI=1.53 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7694/1/artifact/yetus-jdk17-hadoop3-check/output/Dockerfile
GITHUB PR #7694
Optional Tests
uname Linux a163cb1aa4a2 6.14.0-1018-aws #18~24.04.1-Ubuntu SMP Mon Nov 24 19:46:27 UTC 2025 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality dev-support/hbase-personality.sh
git revision branch-2.6 / 7918c59
Max. process+thread count 41 (vs. ulimit of 30000)
modules C: . U: .
Console output https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7694/1/console
versions git=2.34.1 maven=3.9.8
Powered by Apache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase
Copy link

Apache-HBase commented Jan 29, 2026

🎊 +1 overall

Vote Subsystem Runtime Logfile Comment
+0 🆗 reexec 0m 14s Docker mode activated.
-0 ⚠️ yetus 0m 5s Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --author-ignore-list --blanks-eol-ignore-file --blanks-tabs-ignore-file --quick-hadoopcheck
_ Prechecks _
_ branch-2.6 Compile Tests _
_ Patch Compile Tests _
_ Other Tests _
1m 20s
Subsystem Report/Notes
Docker ClientAPI=1.53 ServerAPI=1.53 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7694/1/artifact/yetus-jdk8-hadoop2-check/output/Dockerfile
GITHUB PR #7694
Optional Tests
uname Linux 3d8dddf62578 6.14.0-1018-aws #18~24.04.1-Ubuntu SMP Mon Nov 24 19:46:27 UTC 2025 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality dev-support/hbase-personality.sh
git revision branch-2.6 / 7918c59
Max. process+thread count 29 (vs. ulimit of 30000)
modules C: . U: .
Console output https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7694/1/console
versions git=2.34.1 maven=3.9.8
Powered by Apache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase
Copy link

Apache-HBase commented Jan 29, 2026

💔 -1 overall

Vote Subsystem Runtime Logfile Comment
+0 🆗 reexec 0m 48s Docker mode activated.
_ Prechecks _
+1 💚 dupname 0m 0s No case conflicting files found.
+0 🆗 codespell 0m 0s codespell was not available.
+0 🆗 detsecrets 0m 0s detect-secrets was not available.
+1 💚 @author 0m 0s The patch does not contain any @author tags.
_ branch-2.6 Compile Tests _
+1 💚 mvninstall 3m 26s branch-2.6 passed
-1 ❌ mvnsite 0m 15s /branch-mvnsite-root.txt root in branch-2.6 failed.
+1 💚 spotless 0m 50s branch has no errors when running spotless:check.
_ Patch Compile Tests _
+1 💚 mvninstall 3m 7s the patch passed
+1 💚 blanks 0m 0s The patch has no blanks issues.
+1 💚 mvnsite 3m 27s the patch passed
+1 💚 xmllint 0m 0s No new issues.
+1 💚 spotless 0m 47s patch has no errors when running spotless:check.
_ Other Tests _
+1 💚 asflicense 0m 15s The patch does not generate ASF License warnings.
15m 33s
Subsystem Report/Notes
Docker ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7694/1/artifact/yetus-general-check/output/Dockerfile
GITHUB PR #7694
Optional Tests dupname asflicense mvnsite codespell detsecrets xmllint spotless
uname Linux ec3aca625e8a 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 dev-support/hbase-personality.sh
git revision branch-2.6 / 7918c59
Max. process+thread count 78 (vs. ulimit of 30000)
modules C: . U: .
Console output https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7694/1/console
versions git=2.34.1 maven=3.9.8 xmllint=20913
Powered by Apache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

@PDavid PDavid marked this pull request as ready for review January 29, 2026 16:29
@PDavid PDavid requested a review from Apache9 January 29, 2026 16:29
@apurtell
Copy link
Contributor

apurtell commented Jan 29, 2026

I approved the changes because they are good on their own merit but do not seem to be fixing the site build issues per the precommit report.

[ERROR] Failed to execute goal org.apache.maven.plugins:maven-site-plugin:3.21.0:site (default-site) on project hbase: Execution default-site of goal org.apache.maven.plugins:maven-site-plugin:3.21.0:site failed. NullPointerException -> [Help 1]

I think more needs to be done here before we can claim it is a NPE fix @PDavid

@Umeshkumar9414
Copy link
Contributor

Umeshkumar9414 commented Jan 29, 2026

I did tried it on my local. It didn't give me NPE this time. Let me run it for multiple times.

@PDavid
Copy link
Contributor Author

PDavid commented Jan 29, 2026

Hi @apurtell,

I think mvn site is now successful in this branch, see: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7694/1/artifact/yetus-general-check/output/patch-mvnsite-root.txt

On base branch (branch-2.6) it still fails as expected: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7694/1/artifact/yetus-general-check/output/branch-mvnsite-root.txt

Apache9
Apache9 approved these changes Jan 30, 2026
View reviewed changes
Copy link
Contributor

@Apache9 Apache9 left a comment

Choose a reason for hiding this comment

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

We should also include this for branch-2 and branch-3?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@apurtell apurtell apurtell approved these changes

@Apache9 Apache9 Apache9 approved these changes

+2 more reviewers

@Umeshkumar9414 Umeshkumar9414 Umeshkumar9414 approved these changes

@liuxiaocs7 liuxiaocs7 liuxiaocs7 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@PDavid PDavid

Labels

bug

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@PDavid @Apache-HBase @apurtell @Umeshkumar9414 @Apache9 @liuxiaocs7