Skip to content

HBASE-28676 Make pre commit build for 3.x to only run jdk 17 test - #6001

Merged
Apache9 merged 1 commit into
apache:masterfrom
Apache9:HBASE-28676
Jun 26, 2024
Merged

HBASE-28676 Make pre commit build for 3.x to only run jdk 17 test#6001
Apache9 merged 1 commit into
apache:masterfrom
Apache9:HBASE-28676

Conversation

@Apache9

Copy link
Copy Markdown
Contributor

No description provided.

@Apache9Apache9 self-assigned this Jun 19, 2024
@Apache-HBase

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec0m 25sDocker mode activated.
-0 ⚠️yetus0m 2sUnprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck
_ Prechecks _
_ master Compile Tests _
+0 🆗mvndep0m 19sMaven dependency ordering for branch
+1 💚mvninstall3m 0smaster passed
+1 💚compile1m 58smaster passed
+1 💚shadedjars5m 31sbranch has no errors when building our shaded downstream artifacts.
+1 💚javadoc2m 0smaster passed
_ Patch Compile Tests _
+0 🆗mvndep0m 12sMaven dependency ordering for patch
+1 💚mvninstall2m 49sthe patch passed
+1 💚compile1m 58sthe patch passed
-0 ⚠️javac1m 58sroot generated 75 new + 0 unchanged - 0 fixed = 75 total (was 0)
+1 💚shadedjars5m 24spatch has no errors when building our shaded downstream artifacts.
+1 💚javadoc1m 53sthe patch passed
_ Other Tests _
-1 ❌unit1m 39sroot in the patch failed.
28m 23s
SubsystemReport/Notes
DockerClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6001/1/artifact/yetus-jdk17-hadoop3-check/output/Dockerfile
GITHUB PR#6001
Optional Testsjavac javadoc unit shadedjars compile
unameLinux 0fdbc457f351 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 toolmaven
Personalitydev-support/hbase-personality.sh
git revisionmaster / 046e9d5
Default JavaEclipse Adoptium-17.0.10+7
javachttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6001/1/artifact/yetus-jdk17-hadoop3-check/output/diff-compile-javac-root.txt
unithttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6001/1/artifact/yetus-jdk17-hadoop3-check/output/patch-unit-root.txt
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6001/1/testReport/
Max. process+thread count300 (vs. ulimit of 30000)
modulesC: hbase-build-configuration . U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6001/1/console
versionsgit=2.34.1 maven=3.8.6
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec0m 26sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+0 🆗shelldocs0m 0sShelldocs was not available.
+1 💚@author0m 0sThe patch does not contain any @author tags.
_ master Compile Tests _
+0 🆗mvndep0m 20sMaven dependency ordering for branch
+1 💚mvninstall3m 46smaster passed
-1 ❌compile0m 16sroot in master failed.
+1 💚spotless0m 56sbranch has no errors when running spotless:check.
_ Patch Compile Tests _
+0 🆗mvndep0m 14sMaven dependency ordering for patch
+1 💚mvninstall3m 18sthe patch passed
+1 💚compile8m 27sthe patch passed
-0 ⚠️javac8m 27sroot generated 1206 new + 0 unchanged - 0 fixed = 1206 total (was 0)
+1 💚shellcheck0m 0sThere were no new shellcheck issues.
+1 💚whitespace0m 0sThe patch has no whitespace issues.
-1 ❌xml0m 1sThe patch has 2 ill-formed XML file(s).
+1 💚hadoopcheck5m 7sPatch does not cause any errors with Hadoop 3.3.6.
+1 💚spotless0m 41spatch has no errors when running spotless:check.
_ Other Tests _
+1 💚asflicense0m 18sThe patch does not generate ASF License warnings.
30m 22s
ReasonTests
XMLParsing Error(s):
hbase-build-configuration/pom.xml
pom.xml
SubsystemReport/Notes
DockerClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6001/1/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#6001
Optional Testsdupname asflicense spotless shellcheck shelldocs javac hadoopcheck xml compile
unameLinux a27884dbf7a4 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 toolmaven
Personalitydev-support/hbase-personality.sh
git revisionmaster / 046e9d5
Default JavaEclipse Adoptium-17.0.10+7
compilehttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6001/1/artifact/yetus-general-check/output/branch-compile-root.txt
javachttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6001/1/artifact/yetus-general-check/output/diff-compile-javac-root.txt
xmlhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6001/1/artifact/yetus-general-check/output/xml.txt
Max. process+thread count189 (vs. ulimit of 30000)
modulesC: hbase-build-configuration . U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6001/1/console
versionsgit=2.34.1 maven=3.8.6 shellcheck=0.8.0
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec1m 2sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+0 🆗shelldocs0m 0sShelldocs was not available.
+1 💚@author0m 0sThe patch does not contain any @author tags.
_ master Compile Tests _
+0 🆗mvndep0m 17sMaven dependency ordering for branch
+1 💚mvninstall2m 53smaster passed
-1 ❌compile0m 15sroot in master failed.
+1 💚spotless0m 44sbranch has no errors when running spotless:check.
_ Patch Compile Tests _
+0 🆗mvndep0m 13sMaven dependency ordering for patch
+1 💚mvninstall2m 48sthe patch passed
+1 💚compile7m 42sthe patch passed
-0 ⚠️javac7m 42sroot generated 1179 new + 0 unchanged - 0 fixed = 1179 total (was 0)
+1 💚shellcheck0m 0sThere were no new shellcheck issues.
+1 💚whitespace0m 0sThe patch has no whitespace issues.
-1 ❌xml0m 1sThe patch has 2 ill-formed XML file(s).
+1 💚hadoopcheck4m 56sPatch does not cause any errors with Hadoop 3.3.6.
+1 💚spotless0m 41spatch has no errors when running spotless:check.
_ Other Tests _
+1 💚asflicense0m 22sThe patch does not generate ASF License warnings.
28m 4s
ReasonTests
XMLParsing Error(s):
hbase-build-configuration/pom.xml
pom.xml
SubsystemReport/Notes
DockerClientAPI=1.45 ServerAPI=1.45 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6001/2/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#6001
Optional Testsdupname asflicense spotless shellcheck shelldocs javac hadoopcheck xml compile
unameLinux fe43716f7415 5.4.0-174-generic #193-Ubuntu SMP Thu Mar 7 14:29:28 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionmaster / ff1e52b
Default JavaEclipse Adoptium-17.0.10+7
compilehttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6001/2/artifact/yetus-general-check/output/branch-compile-root.txt
javachttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6001/2/artifact/yetus-general-check/output/diff-compile-javac-root.txt
xmlhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6001/2/artifact/yetus-general-check/output/xml.txt
Max. process+thread count185 (vs. ulimit of 30000)
modulesC: hbase-build-configuration . U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6001/2/console
versionsgit=2.34.1 maven=3.8.6 shellcheck=0.8.0
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@Apache9

Copy link
Copy Markdown
ContributorAuthor

Filed HBASE-28679 for the xml problem, and we can not move releaseTarget to 17 because there are tests failures on mockito, which may require code change. Plan to solve it in HBASE-28675.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec0m 25sDocker mode activated.
-0 ⚠️yetus0m 3sUnprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck
_ Prechecks _
_ master Compile Tests _
+0 🆗mvndep0m 12sMaven dependency ordering for branch
+1 💚mvninstall2m 47smaster passed
+1 💚compile1m 57smaster passed
+1 💚shadedjars5m 25sbranch has no errors when building our shaded downstream artifacts.
+1 💚javadoc1m 56smaster passed
_ Patch Compile Tests _
+0 🆗mvndep0m 12sMaven dependency ordering for patch
+1 💚mvninstall2m 52sthe patch passed
+1 💚compile1m 58sthe patch passed
+1 💚javac1m 58sthe patch passed
+1 💚shadedjars5m 26spatch has no errors when building our shaded downstream artifacts.
+1 💚javadoc1m 56sthe patch passed
_ Other Tests _
+1 💚unit268m 26sroot in the patch passed.
299m 25s
SubsystemReport/Notes
DockerClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6001/2/artifact/yetus-jdk17-hadoop3-check/output/Dockerfile
GITHUB PR#6001
Optional Testsjavac javadoc unit shadedjars compile
unameLinux 100ee231074d 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 toolmaven
Personalitydev-support/hbase-personality.sh
git revisionmaster / ff1e52b
Default JavaEclipse Adoptium-17.0.10+7
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6001/2/testReport/
Max. process+thread count8957 (vs. ulimit of 30000)
modulesC: hbase-build-configuration . U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6001/2/console
versionsgit=2.34.1 maven=3.8.6
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec3m 48sDocker mode activated.
-0 ⚠️yetus0m 3sUnprocessed flag(s): --blanks-eol-ignore-list --blanks-tabs-ignore-list
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+0 🆗codespell0m 0scodespell was not available.
+0 🆗detsecrets0m 0sdetect-secrets was not available.
+0 🆗shelldocs0m 0sShelldocs was not available.
+1 💚@author0m 0sThe patch does not contain any @author tags.
_ master Compile Tests _
+0 🆗mvndep0m 18sMaven dependency ordering for branch
+1 💚mvninstall2m 52smaster passed
-1 ❌compile0m 15s/branch-compile-root.txtroot in master failed.
+1 💚spotless0m 42sbranch has no errors when running spotless:check.
_ Patch Compile Tests _
+0 🆗mvndep0m 13sMaven dependency ordering for patch
+1 💚mvninstall2m 46sthe patch passed
+1 💚compile7m 35sthe patch passed
-0 ⚠️javac7m 35s/results-compile-javac-root.txtroot generated 1179 new + 0 unchanged - 0 fixed = 1179 total (was 0)
+1 💚blanks0m 0sThe patch has no blanks issues.
+1 💚shellcheck0m 0sNo new issues.
+1 💚xmllint0m 0sNo new issues.
+1 💚hadoopcheck4m 57sPatch does not cause any errors with Hadoop 3.3.6.
+1 💚spotless0m 42spatch has no errors when running spotless:check.
_ Other Tests _
+1 💚asflicense0m 17sThe patch does not generate ASF License warnings.
30m 45s
SubsystemReport/Notes
DockerClientAPI=1.45 ServerAPI=1.45 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6001/3/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#6001
Optional Testsdupname asflicense codespell detsecrets shellcheck shelldocs spotless javac xmllint hadoopcheck compile
unameLinux e48bb3803d5c 5.4.0-174-generic #193-Ubuntu SMP Thu Mar 7 14:29:28 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionmaster / 9ad500a
Default JavaEclipse Adoptium-17.0.10+7
Max. process+thread count187 (vs. ulimit of 30000)
modulesC: hbase-build-configuration . U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6001/3/console
versionsgit=2.34.1 maven=3.8.6 shellcheck=0.8.0 xmllint=20913
Powered byApache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

@Apache9

Copy link
Copy Markdown
ContributorAuthor

The error for compile for the current master branch is expected, since we change to use jdk17 for compiling, we need to add some compile args for error prone.

@Apache-HBase

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec0m 27sDocker mode activated.
-0 ⚠️yetus0m 3sUnprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --author-ignore-list --blanks-eol-ignore-list --blanks-tabs-ignore-list --quick-hadoopcheck
_ Prechecks _
_ master Compile Tests _
+0 🆗mvndep0m 31sMaven dependency ordering for branch
+1 💚mvninstall3m 9smaster passed
+1 💚compile1m 58smaster passed
+1 💚javadoc1m 53smaster passed
+1 💚shadedjars5m 37sbranch has no errors when building our shaded downstream artifacts.
_ Patch Compile Tests _
+0 🆗mvndep0m 13sMaven dependency ordering for patch
+1 💚mvninstall2m 50sthe patch passed
+1 💚compile1m 56sthe patch passed
+1 💚javac1m 56sthe patch passed
+1 💚javadoc1m 53sthe patch passed
+1 💚shadedjars5m 28spatch has no errors when building our shaded downstream artifacts.
_ Other Tests _
-1 ❌unit332m 12s/patch-unit-root.txtroot in the patch failed.
412m 39s
SubsystemReport/Notes
DockerClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6001/3/artifact/yetus-jdk17-hadoop3-check/output/Dockerfile
GITHUB PR#6001
Optional Testsjavac javadoc unit shadedjars compile
unameLinux 2a7ec825152b 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 toolmaven
Personalitydev-support/hbase-personality.sh
git revisionmaster / 9ad500a
Default JavaEclipse Adoptium-17.0.10+7
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6001/3/testReport/
Max. process+thread count7336 (vs. ulimit of 30000)
modulesC: hbase-build-configuration . U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6001/3/console
versionsgit=2.34.1 maven=3.8.6
Powered byApache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec0m 47sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+0 🆗codespell0m 0scodespell was not available.
+0 🆗detsecrets0m 0sdetect-secrets was not available.
+0 🆗shelldocs0m 0sShelldocs was not available.
+1 💚@author0m 0sThe patch does not contain any @author tags.
_ master Compile Tests _
+0 🆗mvndep0m 22sMaven dependency ordering for branch
+1 💚mvninstall4m 34smaster passed
-1 ❌compile0m 24s/branch-compile-root.txtroot in master failed.
+1 💚spotless0m 58sbranch has no errors when running spotless:check.
_ Patch Compile Tests _
+0 🆗mvndep0m 18sMaven dependency ordering for patch
+1 💚mvninstall3m 44sthe patch passed
+1 💚compile9m 57sthe patch passed
-0 ⚠️javac9m 57s/results-compile-javac-root.txtroot generated 1179 new + 0 unchanged - 0 fixed = 1179 total (was 0)
+1 💚blanks0m 0sThe patch has no blanks issues.
+1 💚shellcheck0m 0sNo new issues.
+1 💚xmllint0m 0sNo new issues.
+1 💚hadoopcheck6m 41sPatch does not cause any errors with Hadoop 3.3.6.
+1 💚spotless0m 57spatch has no errors when running spotless:check.
_ Other Tests _
+1 💚asflicense0m 25sThe patch does not generate ASF License warnings.
37m 36s
SubsystemReport/Notes
DockerClientAPI=1.45 ServerAPI=1.45 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6001/4/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#6001
Optional Testsdupname asflicense codespell detsecrets shellcheck shelldocs spotless javac xmllint hadoopcheck compile
unameLinux 8fc13d8fa23a 5.4.0-186-generic #206-Ubuntu SMP Fri Apr 26 12:31:10 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionmaster / 407e80f
Default JavaEclipse Adoptium-17.0.10+7
Max. process+thread count186 (vs. ulimit of 30000)
modulesC: hbase-build-configuration . U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6001/4/console
versionsgit=2.34.1 maven=3.8.6 shellcheck=0.8.0 xmllint=20913
Powered byApache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec0m 42sDocker mode activated.
-0 ⚠️yetus0m 3sUnprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --author-ignore-list --blanks-eol-ignore-file --blanks-tabs-ignore-file --quick-hadoopcheck
_ Prechecks _
_ master Compile Tests _
+0 🆗mvndep0m 20sMaven dependency ordering for branch
+1 💚mvninstall3m 41smaster passed
+1 💚compile2m 48smaster passed
+1 💚javadoc2m 29smaster passed
+1 💚shadedjars5m 38sbranch has no errors when building our shaded downstream artifacts.
_ Patch Compile Tests _
+0 🆗mvndep0m 18sMaven dependency ordering for patch
+1 💚mvninstall3m 55sthe patch passed
+1 💚compile2m 47sthe patch passed
+1 💚javac2m 47sthe patch passed
+1 💚javadoc3m 18sthe patch passed
+1 💚shadedjars6m 53spatch has no errors when building our shaded downstream artifacts.
_ Other Tests _
+1 💚unit311m 26sroot in the patch passed.
351m 13s
SubsystemReport/Notes
DockerClientAPI=1.45 ServerAPI=1.45 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6001/4/artifact/yetus-jdk17-hadoop3-check/output/Dockerfile
GITHUB PR#6001
Optional Testsjavac javadoc unit shadedjars compile
unameLinux 741cec0e2f13 5.4.0-186-generic #206-Ubuntu SMP Fri Apr 26 12:31:10 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionmaster / 407e80f
Default JavaEclipse Adoptium-17.0.10+7
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6001/4/testReport/
Max. process+thread count8231 (vs. ulimit of 30000)
modulesC: hbase-build-configuration . U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6001/4/console
versionsgit=2.34.1 maven=3.8.6
Powered byApache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

@Apache9

Copy link
Copy Markdown
ContributorAuthor

OK, good. Xml check can pass now.

@Apache9
Apache9 requested a review from ndimidukJune 24, 2024 01:21
@Apache9

Copy link
Copy Markdown
ContributorAuthor

Merging this will cause the nightly build for master and branch-3 to fail since we still use jdk11 for general checks.

Will start to work on HBASE-28678 ASAP after merging this PR.

Comment threadpom.xml
<tar.name>${project.build.finalName}.tar.gz</tar.name>
<maven.build.timestamp.format>yyyy-MM-dd'T'HH:mm</maven.build.timestamp.format>
<buildDate>${maven.build.timestamp}</buildDate>
<compileSource>1.8</compileSource>

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.

Big jump!

Comment threadpom.xml
<buildDate>${maven.build.timestamp}</buildDate>
<compileSource>1.8</compileSource>
<compileSource>17</compileSource>
<releaseTarget>8</releaseTarget>

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.

What does it mean to have compile target at 17 but release target at 8? Can javac emit bytecode for 17 features for an 8 VM?

We are dropping JDK8 runtime support, so I think we can move the release target up as well.

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

ReleaseTarget 17 will break mockito, need to dig more...

@Apache9

Copy link
Copy Markdown
ContributorAuthor

Filed HBASE-28679 for the xml problem, and we can not move releaseTarget to 17 because there are tests failures on mockito, which may require code change. Plan to solve it in HBASE-28675.

@ndimiduk See here, I tried to jump to 17 too for releaseTarget at first, but it breaks mockito, so I plan to address it in HBASE-28675 since it may require java code changes.

@Apache9
Apache9 merged commit 80516d7 into apache:masterJun 26, 2024
Apache9 added a commit that referenced this pull request Jun 26, 2024
)
Signed-off-by: Nick Dimiduk <ndimiduk@apache.org>
(cherry picked from commit 80516d7)
@ndimiduk

Copy link
Copy Markdown
Member

Makes sense. Thanks @Apache9 !

prabhjyotsingh pushed a commit to acceldata-io/hbase that referenced this pull request Jun 9, 2026
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.

3 participants

@Apache9@Apache-HBase@ndimiduk