Skip to content

Backport "HBASE-26622 Update error-prone to 2.10" to branch-2 - #4150

Merged
ndimiduk merged 1 commit into
apache:branch-2from
ndimiduk:26622-upgrade-errorprone-2.10-branch-2
Mar 3, 2022
Merged

Backport "HBASE-26622 Update error-prone to 2.10" to branch-2#4150
ndimiduk merged 1 commit into
apache:branch-2from
ndimiduk:26622-upgrade-errorprone-2.10-branch-2

Conversation

@ndimiduk

@ndimidukndimiduk commented Mar 2, 2022

Copy link
Copy Markdown
Member

Author: Mike Drob mdrob@apache.org
Co-authored-by: Nick Dimiduk ndimiduk@apache.org
Signed-off-by: Andrew Purtell apurtell@apache.org

@ndimidukndimiduk added the backport This PR is a back port of some issue or issues already committed to master label Mar 2, 2022
@Apache-HBase

This comment was marked as outdated.

Sign in to view
@Apache-HBase

This comment was marked as outdated.

Sign in to view
@Apache-HBase

This comment was marked as outdated.

Sign in to view
@madrob@ndimiduk
HBASE-26622 Update error-prone to 2.10
05c0157
Author: Mike Drob <mdrob@apache.org>
Co-authored-by: Nick Dimiduk <ndimiduk@apache.org>
Signed-off-by: Andrew Purtell <apurtell@apache.org>
@ndimiduk
ndimidukforce-pushed the 26622-upgrade-errorprone-2.10-branch-2 branch from 5be3f6f to 05c0157CompareMarch 3, 2022 10:23
@ndimiduk
ndimiduk merged commit 8a9e083 into apache:branch-2Mar 3, 2022
@ndimiduk
ndimiduk deleted the 26622-upgrade-errorprone-2.10-branch-2 branch March 3, 2022 10:52
@Apache-HBase

Apache-HBase commented Mar 3, 2022

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec1m 32sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+1 💚hbaseanti0m 0sPatch does not have any anti-patterns.
+1 💚@author0m 0sThe patch does not contain any @author tags.
_ branch-2 Compile Tests _
+0 🆗mvndep0m 15sMaven dependency ordering for branch
+1 💚mvninstall2m 41sbranch-2 passed
+1 💚compile7m 9sbranch-2 passed
+1 💚checkstyle1m 20sbranch-2 passed
+1 💚spotbugs12m 4sbranch-2 passed
_ Patch Compile Tests _
+0 🆗mvndep0m 12sMaven dependency ordering for patch
+1 💚mvninstall2m 40sthe patch passed
+1 💚compile7m 47sthe patch passed
-0 ⚠️javac7m 47sroot generated 118 new + 1552 unchanged - 329 fixed = 1670 total (was 1881)
+1 💚checkstyle1m 14sroot: The patch generated 0 new + 81 unchanged - 5 fixed = 81 total (was 86)
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚xml0m 0sThe patch has no ill-formed XML file.
+1 💚hadoopcheck8m 46sPatch does not cause any errors with Hadoop 3.1.2 3.2.1.
+1 💚spotbugs13m 52sthe patch passed
_ Other Tests _
+1 💚asflicense0m 54sThe patch does not generate ASF License warnings.
67m 8s
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4150/2/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#4150
Optional Testsdupname asflicense javac spotbugs hadoopcheck hbaseanti checkstyle compile xml
unameLinux d7520c7c4b75 5.4.0-90-generic #101-Ubuntu SMP Fri Oct 15 20:00:55 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionbranch-2 / d020cba
Default JavaAdoptOpenJDK-1.8.0_282-b08
javachttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4150/2/artifact/yetus-general-check/output/diff-compile-javac-root.txt
Max. process+thread count89 (vs. ulimit of 12500)
modulesC: hbase-common hbase-asyncfs hbase-server hbase-mapreduce hbase-thrift hbase-endpoint . U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4150/2/console
versionsgit=2.17.1 maven=3.6.3 spotbugs=4.2.2
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Apache-HBase commented Mar 3, 2022

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec1m 28sDocker mode activated.
-0 ⚠️yetus0m 5sUnprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck
_ Prechecks _
_ branch-2 Compile Tests _
+0 🆗mvndep0m 17sMaven dependency ordering for branch
+1 💚mvninstall3m 10sbranch-2 passed
+1 💚compile2m 16sbranch-2 passed
+1 💚shadedjars4m 31sbranch has no errors when building our shaded downstream artifacts.
+1 💚javadoc4m 38sbranch-2 passed
_ Patch Compile Tests _
+0 🆗mvndep0m 16sMaven dependency ordering for patch
+1 💚mvninstall3m 24sthe patch passed
+1 💚compile2m 13sthe patch passed
+1 💚javac2m 13sthe patch passed
+1 💚shadedjars3m 56spatch has no errors when building our shaded downstream artifacts.
+1 💚javadoc4m 16sthe patch passed
_ Other Tests _
+1 💚unit418m 56sroot in the patch passed.
452m 0s
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4150/2/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile
GITHUB PR#4150
Optional Testsjavac javadoc unit shadedjars compile
unameLinux 6239237511f4 5.4.0-90-generic #101-Ubuntu SMP Fri Oct 15 20:00:55 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionbranch-2 / d020cba
Default JavaAdoptOpenJDK-11.0.10+9
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4150/2/testReport/
Max. process+thread count2906 (vs. ulimit of 12500)
modulesC: hbase-common hbase-asyncfs hbase-server hbase-mapreduce hbase-thrift hbase-endpoint . U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4150/2/console
versionsgit=2.17.1 maven=3.6.3
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Apache-HBase commented Mar 3, 2022

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec2m 49sDocker mode activated.
-0 ⚠️yetus0m 5sUnprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck
_ Prechecks _
_ branch-2 Compile Tests _
+0 🆗mvndep0m 16sMaven dependency ordering for branch
+1 💚mvninstall2m 19sbranch-2 passed
+1 💚compile1m 32sbranch-2 passed
+1 💚shadedjars3m 10sbranch has no errors when building our shaded downstream artifacts.
+1 💚javadoc2m 57sbranch-2 passed
_ Patch Compile Tests _
+0 🆗mvndep0m 14sMaven dependency ordering for patch
+1 💚mvninstall2m 11sthe patch passed
+1 💚compile1m 33sthe patch passed
+1 💚javac1m 33sthe patch passed
+1 💚shadedjars3m 7spatch has no errors when building our shaded downstream artifacts.
+1 💚javadoc2m 56sthe patch passed
_ Other Tests _
+1 💚unit507m 18sroot in the patch passed.
533m 6s
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4150/2/artifact/yetus-jdk8-hadoop2-check/output/Dockerfile
GITHUB PR#4150
Optional Testsjavac javadoc unit shadedjars compile
unameLinux c1f9f2274ce2 5.4.0-96-generic #109-Ubuntu SMP Wed Jan 12 16:49:16 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionbranch-2 / d020cba
Default JavaAdoptOpenJDK-1.8.0_282-b08
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4150/2/testReport/
Max. process+thread count2379 (vs. ulimit of 12500)
modulesC: hbase-common hbase-asyncfs hbase-server hbase-mapreduce hbase-thrift hbase-endpoint . U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4150/2/console
versionsgit=2.17.1 maven=3.6.3
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

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

Reviewers

No reviews

Assignees

No one assigned

Labels

backportThis PR is a back port of some issue or issues already committed to master

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@ndimiduk@Apache-HBase@madrob