You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@hbase.apache.org by GitBox <gi...@apache.org> on 2020/03/10 16:28:38 UTC
[GitHub] [hbase] Apache9 opened a new pull request #1265: HBASE-23077 move
entirely to spotbugs
Apache9 opened a new pull request #1265: HBASE-23077 move entirely to spotbugs
URL: https://github.com/apache/hbase/pull/1265
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [hbase] Apache-HBase commented on issue #1265: HBASE-23077 move
entirely to spotbugs
Posted by GitBox <gi...@apache.org>.
Apache-HBase commented on issue #1265: HBASE-23077 move entirely to spotbugs
URL: https://github.com/apache/hbase/pull/1265#issuecomment-597393962
(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/2/console in case of problems.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [hbase] Apache-HBase commented on issue #1265: HBASE-23077 move
entirely to spotbugs
Posted by GitBox <gi...@apache.org>.
Apache-HBase commented on issue #1265: HBASE-23077 move entirely to spotbugs
URL: https://github.com/apache/hbase/pull/1265#issuecomment-597970444
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Comment |
|:----:|----------:|--------:|:--------|
| +0 :ok: | reexec | 4m 19s | Docker mode activated. |
||| _ Prechecks _ |
| +1 :green_heart: | dupname | 0m 0s | No case conflicting files found. |
| +0 :ok: | shelldocs | 0m 0s | Shelldocs was not available. |
| +1 :green_heart: | @author | 0m 1s | The patch does not contain any @author tags. |
||| _ master Compile Tests _ |
| +0 :ok: | mvndep | 0m 41s | Maven dependency ordering for branch |
| +1 :green_heart: | mvninstall | 8m 43s | master passed |
| -1 :x: | spotbugs | 0m 18s | hbase-protocol-shaded in master failed. |
| -1 :x: | spotbugs | 0m 12s | hbase-common in master failed. |
| -1 :x: | spotbugs | 0m 14s | hbase-client in master failed. |
| -1 :x: | spotbugs | 0m 13s | hbase-zookeeper in master failed. |
| -1 :x: | spotbugs | 0m 12s | hbase-http in master failed. |
| -1 :x: | spotbugs | 0m 14s | hbase-procedure in master failed. |
| -1 :x: | spotbugs | 0m 13s | hbase-server in master failed. |
| -1 :x: | spotbugs | 0m 12s | hbase-mapreduce in master failed. |
| -1 :x: | spotbugs | 0m 14s | hbase-thrift in master failed. |
| -1 :x: | spotbugs | 0m 14s | hbase-rest in master failed. |
| -1 :x: | spotbugs | 0m 12s | hbase-examples in master failed. |
| -1 :x: | spotbugs | 0m 12s | hbase-hbtop in master failed. |
| -1 :x: | spotbugs | 0m 13s | hbase-archetypes in master failed. |
| -1 :x: | spotbugs | 0m 12s | root in master failed. |
||| _ Patch Compile Tests _ |
| +0 :ok: | mvndep | 0m 22s | Maven dependency ordering for patch |
| +1 :green_heart: | mvninstall | 7m 20s | the patch passed |
| +1 :green_heart: | hadolint | 0m 3s | There were no new hadolint issues. |
| +1 :green_heart: | shellcheck | 0m 2s | There were no new shellcheck issues. |
| +1 :green_heart: | whitespace | 0m 0s | The patch has no whitespace issues. |
| +1 :green_heart: | xml | 0m 34s | The patch has no ill-formed XML file. |
| +1 :green_heart: | hadoopcheck | 23m 10s | Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2. |
| +1 :green_heart: | spotbugs | 33m 44s | the patch passed |
||| _ Other Tests _ |
| +1 :green_heart: | asflicense | 4m 9s | The patch does not generate ASF License warnings. |
| | | 98m 42s | |
| Subsystem | Report/Notes |
|----------:|:-------------|
| Docker | Client=19.03.8 Server=19.03.8 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/8/artifact/yetus-general-check/output/Dockerfile |
| GITHUB PR | https://github.com/apache/hbase/pull/1265 |
| Optional Tests | dupname asflicense hadolint shellcheck shelldocs xml spotbugs hadoopcheck |
| uname | Linux 6501e005d232 4.15.0-74-generic #84-Ubuntu SMP Thu Dec 19 08:06:28 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux |
| Build tool | maven |
| Personality | dev-support/hbase-personality.sh |
| git revision | master / cf121227aa |
| spotbugs | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/8/artifact/yetus-general-check/output/branch-spotbugs-hbase-protocol-shaded.txt |
| spotbugs | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/8/artifact/yetus-general-check/output/branch-spotbugs-hbase-common.txt |
| spotbugs | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/8/artifact/yetus-general-check/output/branch-spotbugs-hbase-client.txt |
| spotbugs | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/8/artifact/yetus-general-check/output/branch-spotbugs-hbase-zookeeper.txt |
| spotbugs | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/8/artifact/yetus-general-check/output/branch-spotbugs-hbase-http.txt |
| spotbugs | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/8/artifact/yetus-general-check/output/branch-spotbugs-hbase-procedure.txt |
| spotbugs | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/8/artifact/yetus-general-check/output/branch-spotbugs-hbase-server.txt |
| spotbugs | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/8/artifact/yetus-general-check/output/branch-spotbugs-hbase-mapreduce.txt |
| spotbugs | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/8/artifact/yetus-general-check/output/branch-spotbugs-hbase-thrift.txt |
| spotbugs | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/8/artifact/yetus-general-check/output/branch-spotbugs-hbase-rest.txt |
| spotbugs | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/8/artifact/yetus-general-check/output/branch-spotbugs-hbase-examples.txt |
| spotbugs | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/8/artifact/yetus-general-check/output/branch-spotbugs-hbase-hbtop.txt |
| spotbugs | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/8/artifact/yetus-general-check/output/branch-spotbugs-hbase-archetypes.txt |
| spotbugs | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/8/artifact/yetus-general-check/output/branch-spotbugs-root.txt |
| Max. process+thread count | 84 (vs. ulimit of 10000) |
| modules | C: hbase-protocol-shaded hbase-common hbase-client hbase-zookeeper hbase-http hbase-procedure hbase-server hbase-mapreduce hbase-thrift hbase-shell hbase-rest hbase-examples hbase-shaded hbase-shaded/hbase-shaded-client hbase-shaded/hbase-shaded-client-byo-hadoop hbase-hbtop hbase-shaded/hbase-shaded-testing-util hbase-shaded/hbase-shaded-check-invariants hbase-shaded/hbase-shaded-with-hadoop-check-invariants hbase-archetypes . U: . |
| Console output | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/8/console |
| versions | git=2.17.1 maven=2018-06-17T18:33:14Z) shellcheck=0.4.6 hadolint=1.17.5-0-g443423c |
| Powered by | Apache Yetus 0.11.1 https://yetus.apache.org |
This message was automatically generated.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [hbase] Apache-HBase commented on issue #1265: HBASE-23077 move
entirely to spotbugs
Posted by GitBox <gi...@apache.org>.
Apache-HBase commented on issue #1265: HBASE-23077 move entirely to spotbugs
URL: https://github.com/apache/hbase/pull/1265#issuecomment-597428551
(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/3/console in case of problems.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [hbase] Apache-HBase commented on issue #1265: HBASE-23077 move
entirely to spotbugs
Posted by GitBox <gi...@apache.org>.
Apache-HBase commented on issue #1265: HBASE-23077 move entirely to spotbugs
URL: https://github.com/apache/hbase/pull/1265#issuecomment-597440116
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Comment |
|:----:|----------:|--------:|:--------|
| +0 :ok: | reexec | 3m 27s | Docker mode activated. |
| -0 :warning: | yetus | 0m 3s | Unprocessed flag(s): --brief-report-file --findbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck |
||| _ Prechecks _ |
||| _ master Compile Tests _ |
| +0 :ok: | mvndep | 0m 48s | Maven dependency ordering for branch |
| +1 :green_heart: | mvninstall | 6m 5s | master passed |
| +1 :green_heart: | compile | 3m 27s | master passed |
| +1 :green_heart: | shadedjars | 5m 20s | branch has no errors when building our shaded downstream artifacts. |
| +1 :green_heart: | javadoc | 8m 53s | master passed |
||| _ Patch Compile Tests _ |
| +0 :ok: | mvndep | 0m 22s | Maven dependency ordering for patch |
| +1 :green_heart: | mvninstall | 5m 55s | the patch passed |
| +1 :green_heart: | compile | 3m 25s | the patch passed |
| +1 :green_heart: | javac | 3m 25s | the patch passed |
| +1 :green_heart: | shadedjars | 5m 18s | patch has no errors when building our shaded downstream artifacts. |
| +1 :green_heart: | javadoc | 8m 48s | the patch passed |
||| _ Other Tests _ |
| -1 :x: | unit | 1m 40s | root in the patch failed. |
| | | 57m 9s | |
| Subsystem | Report/Notes |
|----------:|:-------------|
| Docker | Client=19.03.7 Server=19.03.7 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/3/artifact/yetus-jdk8-hadoop2-check/output/Dockerfile |
| GITHUB PR | https://github.com/apache/hbase/pull/1265 |
| Optional Tests | javac javadoc unit shadedjars compile |
| uname | Linux ee296e14af3e 4.15.0-74-generic #84-Ubuntu SMP Thu Dec 19 08:06:28 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux |
| Build tool | maven |
| Personality | dev-support/hbase-personality.sh |
| git revision | master / e345a95461 |
| Default Java | 1.8.0_232 |
| unit | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/3/artifact/yetus-jdk8-hadoop2-check/output/patch-unit-root.txt |
| Test Results | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/3/testReport/ |
| Max. process+thread count | 382 (vs. ulimit of 10000) |
| modules | C: hbase-protocol-shaded hbase-common hbase-client hbase-zookeeper hbase-http hbase-procedure hbase-server hbase-mapreduce hbase-thrift hbase-shell hbase-rest hbase-examples hbase-shaded hbase-shaded/hbase-shaded-client hbase-shaded/hbase-shaded-client-byo-hadoop hbase-hbtop hbase-shaded/hbase-shaded-testing-util hbase-shaded/hbase-shaded-check-invariants hbase-shaded/hbase-shaded-with-hadoop-check-invariants hbase-archetypes . U: . |
| Console output | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/3/console |
| versions | git=2.17.1 maven=2018-06-17T18:33:14Z) |
| Powered by | Apache Yetus 0.11.1 https://yetus.apache.org |
This message was automatically generated.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [hbase] Apache-HBase commented on issue #1265: HBASE-23077 move
entirely to spotbugs
Posted by GitBox <gi...@apache.org>.
Apache-HBase commented on issue #1265: HBASE-23077 move entirely to spotbugs
URL: https://github.com/apache/hbase/pull/1265#issuecomment-597467543
(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/4/console in case of problems.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [hbase] Apache-HBase commented on issue #1265: HBASE-23077 move
entirely to spotbugs
Posted by GitBox <gi...@apache.org>.
Apache-HBase commented on issue #1265: HBASE-23077 move entirely to spotbugs
URL: https://github.com/apache/hbase/pull/1265#issuecomment-597513136
:confetti_ball: **+1 overall**
| Vote | Subsystem | Runtime | Comment |
|:----:|----------:|--------:|:--------|
| +0 :ok: | reexec | 1m 52s | Docker mode activated. |
| -0 :warning: | yetus | 0m 4s | Unprocessed flag(s): --brief-report-file --findbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck |
||| _ Prechecks _ |
||| _ master Compile Tests _ |
| +0 :ok: | mvndep | 0m 46s | Maven dependency ordering for branch |
| +1 :green_heart: | mvninstall | 7m 25s | master passed |
| +1 :green_heart: | compile | 4m 27s | master passed |
| +1 :green_heart: | shadedjars | 5m 9s | branch has no errors when building our shaded downstream artifacts. |
| -0 :warning: | javadoc | 0m 18s | hbase-common in master failed. |
| -0 :warning: | javadoc | 0m 24s | hbase-client in master failed. |
| -0 :warning: | javadoc | 0m 16s | hbase-zookeeper in master failed. |
| -0 :warning: | javadoc | 0m 17s | hbase-http in master failed. |
| -0 :warning: | javadoc | 0m 16s | hbase-procedure in master failed. |
| -0 :warning: | javadoc | 0m 16s | hbase-hbtop in master failed. |
| -0 :warning: | javadoc | 0m 16s | hbase-archetypes in master failed. |
| -0 :warning: | javadoc | 0m 18s | root in master failed. |
| -0 :warning: | javadoc | 0m 19s | hbase-examples in master failed. |
| -0 :warning: | javadoc | 0m 20s | hbase-mapreduce in master failed. |
| -0 :warning: | javadoc | 0m 23s | hbase-rest in master failed. |
| -0 :warning: | javadoc | 0m 39s | hbase-server in master failed. |
| -0 :warning: | javadoc | 0m 46s | hbase-thrift in master failed. |
||| _ Patch Compile Tests _ |
| +0 :ok: | mvndep | 0m 21s | Maven dependency ordering for patch |
| +1 :green_heart: | mvninstall | 6m 15s | the patch passed |
| +1 :green_heart: | compile | 4m 1s | the patch passed |
| +1 :green_heart: | javac | 4m 1s | the patch passed |
| +1 :green_heart: | shadedjars | 5m 2s | patch has no errors when building our shaded downstream artifacts. |
| -0 :warning: | javadoc | 0m 17s | hbase-common in the patch failed. |
| -0 :warning: | javadoc | 0m 25s | hbase-client in the patch failed. |
| -0 :warning: | javadoc | 0m 16s | hbase-zookeeper in the patch failed. |
| -0 :warning: | javadoc | 0m 16s | hbase-http in the patch failed. |
| -0 :warning: | javadoc | 0m 17s | hbase-procedure in the patch failed. |
| -0 :warning: | javadoc | 0m 40s | hbase-server in the patch failed. |
| -0 :warning: | javadoc | 0m 20s | hbase-mapreduce in the patch failed. |
| -0 :warning: | javadoc | 0m 47s | hbase-thrift in the patch failed. |
| -0 :warning: | javadoc | 0m 22s | hbase-rest in the patch failed. |
| -0 :warning: | javadoc | 0m 20s | hbase-examples in the patch failed. |
| -0 :warning: | javadoc | 0m 17s | hbase-hbtop in the patch failed. |
| -0 :warning: | javadoc | 0m 15s | hbase-archetypes in the patch failed. |
| -0 :warning: | javadoc | 0m 19s | root in the patch failed. |
||| _ Other Tests _ |
| -0 :warning: | unit | 81m 3s | root in the patch failed. |
| | | 135m 20s | |
| Subsystem | Report/Notes |
|----------:|:-------------|
| Docker | Client=19.03.7 Server=19.03.7 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/4/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile |
| GITHUB PR | https://github.com/apache/hbase/pull/1265 |
| Optional Tests | javac javadoc unit shadedjars compile |
| uname | Linux f2482a555fcd 4.15.0-60-generic #67-Ubuntu SMP Thu Aug 22 16:55:30 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux |
| Build tool | maven |
| Personality | dev-support/hbase-personality.sh |
| git revision | master / d019fd2eed |
| Default Java | 2020-01-14 |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/4/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-common.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/4/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-client.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/4/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-zookeeper.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/4/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-http.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/4/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-procedure.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/4/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-hbtop.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/4/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-archetypes.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/4/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-root.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/4/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-examples.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/4/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-mapreduce.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/4/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-rest.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/4/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-server.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/4/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-thrift.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/4/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-common.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/4/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-client.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/4/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-zookeeper.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/4/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-http.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/4/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-procedure.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/4/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-server.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/4/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-mapreduce.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/4/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-thrift.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/4/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-rest.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/4/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-examples.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/4/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-hbtop.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/4/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-archetypes.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/4/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-root.txt |
| unit | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/4/artifact/yetus-jdk11-hadoop3-check/output/patch-unit-root.txt |
| Test Results | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/4/testReport/ |
| Max. process+thread count | 9658 (vs. ulimit of 10000) |
| modules | C: hbase-protocol-shaded hbase-common hbase-client hbase-zookeeper hbase-http hbase-procedure hbase-server hbase-mapreduce hbase-thrift hbase-shell hbase-rest hbase-examples hbase-shaded hbase-shaded/hbase-shaded-client hbase-shaded/hbase-shaded-client-byo-hadoop hbase-hbtop hbase-shaded/hbase-shaded-testing-util hbase-shaded/hbase-shaded-check-invariants hbase-shaded/hbase-shaded-with-hadoop-check-invariants hbase-archetypes . U: . |
| Console output | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/4/console |
| versions | git=2.17.1 maven=2018-06-17T18:33:14Z) |
| Powered by | Apache Yetus 0.11.1 https://yetus.apache.org |
This message was automatically generated.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [hbase] Apache9 commented on issue #1265: HBASE-23077 move entirely
to spotbugs
Posted by GitBox <gi...@apache.org>.
Apache9 commented on issue #1265: HBASE-23077 move entirely to spotbugs
URL: https://github.com/apache/hbase/pull/1265#issuecomment-597643773
Skip more modules in hbase-personality as hbase-shaded* do not contain any java code.
Add spotbugs plugin declaration directly in hbase-client, let's see if spotbugs for hbase-client could pass.
Add special config maxHeap in hbase-server to see if we still have OOM.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [hbase] Apache-HBase commented on issue #1265: HBASE-23077 move
entirely to spotbugs
Posted by GitBox <gi...@apache.org>.
Apache-HBase commented on issue #1265: HBASE-23077 move entirely to spotbugs
URL: https://github.com/apache/hbase/pull/1265#issuecomment-597975693
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Comment |
|:----:|----------:|--------:|:--------|
| +0 :ok: | reexec | 1m 44s | Docker mode activated. |
| -0 :warning: | yetus | 0m 3s | Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck |
||| _ Prechecks _ |
||| _ master Compile Tests _ |
| +0 :ok: | mvndep | 0m 47s | Maven dependency ordering for branch |
| +1 :green_heart: | mvninstall | 5m 30s | master passed |
| +1 :green_heart: | compile | 3m 11s | master passed |
| +1 :green_heart: | shadedjars | 4m 53s | branch has no errors when building our shaded downstream artifacts. |
| +1 :green_heart: | javadoc | 8m 15s | master passed |
||| _ Patch Compile Tests _ |
| +0 :ok: | mvndep | 0m 20s | Maven dependency ordering for patch |
| +1 :green_heart: | mvninstall | 5m 12s | the patch passed |
| +1 :green_heart: | compile | 3m 4s | the patch passed |
| +1 :green_heart: | javac | 3m 4s | the patch passed |
| +1 :green_heart: | shadedjars | 4m 43s | patch has no errors when building our shaded downstream artifacts. |
| +1 :green_heart: | javadoc | 8m 25s | the patch passed |
||| _ Other Tests _ |
| -1 :x: | unit | 70m 46s | root in the patch failed. |
| | | 122m 5s | |
| Subsystem | Report/Notes |
|----------:|:-------------|
| Docker | Client=19.03.8 Server=19.03.8 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/8/artifact/yetus-jdk8-hadoop2-check/output/Dockerfile |
| GITHUB PR | https://github.com/apache/hbase/pull/1265 |
| Optional Tests | javac javadoc unit shadedjars compile |
| uname | Linux 16c32ffd04c6 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux |
| Build tool | maven |
| Personality | dev-support/hbase-personality.sh |
| git revision | master / cf121227aa |
| Default Java | 1.8.0_232 |
| unit | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/8/artifact/yetus-jdk8-hadoop2-check/output/patch-unit-root.txt |
| Test Results | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/8/testReport/ |
| Max. process+thread count | 6013 (vs. ulimit of 10000) |
| modules | C: hbase-protocol-shaded hbase-common hbase-client hbase-zookeeper hbase-http hbase-procedure hbase-server hbase-mapreduce hbase-thrift hbase-shell hbase-rest hbase-examples hbase-shaded hbase-shaded/hbase-shaded-client hbase-shaded/hbase-shaded-client-byo-hadoop hbase-hbtop hbase-shaded/hbase-shaded-testing-util hbase-shaded/hbase-shaded-check-invariants hbase-shaded/hbase-shaded-with-hadoop-check-invariants hbase-archetypes . U: . |
| Console output | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/8/console |
| versions | git=2.17.1 maven=2018-06-17T18:33:14Z) |
| Powered by | Apache Yetus 0.11.1 https://yetus.apache.org |
This message was automatically generated.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [hbase] Apache9 commented on issue #1265: HBASE-23077 move entirely
to spotbugs
Posted by GitBox <gi...@apache.org>.
Apache9 commented on issue #1265: HBASE-23077 move entirely to spotbugs
URL: https://github.com/apache/hbase/pull/1265#issuecomment-597404664
OK, let me check what's wrong with the new Dockerfile...
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [hbase] busbey commented on a change in pull request #1265:
HBASE-23077 move entirely to spotbugs
Posted by GitBox <gi...@apache.org>.
busbey commented on a change in pull request #1265: HBASE-23077 move entirely to spotbugs
URL: https://github.com/apache/hbase/pull/1265#discussion_r390453294
##########
File path: hbase-shaded/hbase-shaded-check-invariants/pom.xml
##########
@@ -118,8 +118,9 @@
<exclude>log4j:*</exclude>
<exclude>commons-logging:*</exclude>
<!-- annotations that never change -->
- <exclude>com.google.code.findbugs:*</exclude>
- <exclude>com.github.stephenc.findbugs:*</exclude>
+ <exclude>com.google.code.findbugs:*</exclude>
Review comment:
nit: whitespace error?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [hbase] ndimiduk commented on a change in pull request #1265:
HBASE-23077 move entirely to spotbugs
Posted by GitBox <gi...@apache.org>.
ndimiduk commented on a change in pull request #1265: HBASE-23077 move entirely to spotbugs
URL: https://github.com/apache/hbase/pull/1265#discussion_r391722015
##########
File path: hbase-shaded/hbase-shaded-testing-util/pom.xml
##########
@@ -184,7 +184,8 @@
<exclude>org.apache.hbase:hbase-resource-bundle</exclude>
<exclude>org.slf4j:*</exclude>
<exclude>com.google.code.findbugs:*</exclude>
- <exclude>com.github.stephenc.findbugs:*</exclude>
+ <exclude>com.github.stephenc.fingbugs:*</exclude>
Review comment:
oops, copy-pasted everywhere.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [hbase] Apache9 commented on issue #1265: HBASE-23077 move entirely
to spotbugs
Posted by GitBox <gi...@apache.org>.
Apache9 commented on issue #1265: HBASE-23077 move entirely to spotbugs
URL: https://github.com/apache/hbase/pull/1265#issuecomment-597620353
OK, at least we got the spotbugs works. Let me check why they fail.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [hbase] Apache9 commented on issue #1265: HBASE-23077 move entirely
to spotbugs
Posted by GitBox <gi...@apache.org>.
Apache9 commented on issue #1265: HBASE-23077 move entirely to spotbugs
URL: https://github.com/apache/hbase/pull/1265#issuecomment-597945464
@busbey The patch is ready. PTAL.
Thanks.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [hbase] Apache-HBase commented on issue #1265: HBASE-23077 move
entirely to spotbugs
Posted by GitBox <gi...@apache.org>.
Apache-HBase commented on issue #1265: HBASE-23077 move entirely to spotbugs
URL: https://github.com/apache/hbase/pull/1265#issuecomment-597428557
(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/3/console in case of problems.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [hbase] busbey commented on a change in pull request #1265:
HBASE-23077 move entirely to spotbugs
Posted by GitBox <gi...@apache.org>.
busbey commented on a change in pull request #1265: HBASE-23077 move entirely to spotbugs
URL: https://github.com/apache/hbase/pull/1265#discussion_r390453844
##########
File path: hbase-shaded/pom.xml
##########
@@ -146,8 +146,9 @@
hadoop-including module -->
<exclude>org.apache.hbase:hbase-resource-bundle</exclude>
<exclude>org.slf4j:*</exclude>
- <exclude>com.google.code.findbugs:*</exclude>
- <exclude>com.github.stephenc.findbugs:*</exclude>
+ <exclude>com.google.code.findbugs:*</exclude>
Review comment:
nit: whitespace error
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [hbase] Apache-HBase commented on issue #1265: HBASE-23077 move
entirely to spotbugs
Posted by GitBox <gi...@apache.org>.
Apache-HBase commented on issue #1265: HBASE-23077 move entirely to spotbugs
URL: https://github.com/apache/hbase/pull/1265#issuecomment-597287676
:confetti_ball: **+1 overall**
| Vote | Subsystem | Runtime | Comment |
|:----:|----------:|--------:|:--------|
| +0 :ok: | reexec | 1m 29s | Docker mode activated. |
| -0 :warning: | yetus | 0m 3s | Unprocessed flag(s): --brief-report-file --findbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck |
||| _ Prechecks _ |
||| _ master Compile Tests _ |
| +0 :ok: | mvndep | 0m 45s | Maven dependency ordering for branch |
| +1 :green_heart: | mvninstall | 6m 7s | master passed |
| +1 :green_heart: | compile | 3m 25s | master passed |
| +1 :green_heart: | shadedjars | 5m 16s | branch has no errors when building our shaded downstream artifacts. |
| +1 :green_heart: | javadoc | 8m 59s | master passed |
||| _ Patch Compile Tests _ |
| +0 :ok: | mvndep | 0m 20s | Maven dependency ordering for patch |
| +1 :green_heart: | mvninstall | 5m 58s | the patch passed |
| +1 :green_heart: | compile | 3m 25s | the patch passed |
| +1 :green_heart: | javac | 3m 25s | the patch passed |
| +1 :green_heart: | shadedjars | 5m 25s | patch has no errors when building our shaded downstream artifacts. |
| +1 :green_heart: | javadoc | 9m 22s | the patch passed |
||| _ Other Tests _ |
| +1 :green_heart: | unit | 159m 8s | root in the patch passed. |
| | | 214m 56s | |
| Subsystem | Report/Notes |
|----------:|:-------------|
| Docker | Client=19.03.7 Server=19.03.7 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/1/artifact/yetus-jdk8-hadoop2-check/output/Dockerfile |
| GITHUB PR | https://github.com/apache/hbase/pull/1265 |
| Optional Tests | javac javadoc unit shadedjars compile |
| uname | Linux 8ea7a0d9b510 4.15.0-74-generic #84-Ubuntu SMP Thu Dec 19 08:06:28 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux |
| Build tool | maven |
| Personality | dev-support/hbase-personality.sh |
| git revision | master / 31484f007f |
| Default Java | 1.8.0_232 |
| Test Results | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/1/testReport/ |
| Max. process+thread count | 6828 (vs. ulimit of 10000) |
| modules | C: hbase-protocol-shaded hbase-common hbase-client hbase-zookeeper hbase-http hbase-procedure hbase-server hbase-mapreduce hbase-thrift hbase-shell hbase-rest hbase-examples hbase-shaded hbase-shaded/hbase-shaded-client hbase-shaded/hbase-shaded-client-byo-hadoop hbase-hbtop hbase-shaded/hbase-shaded-testing-util hbase-shaded/hbase-shaded-check-invariants hbase-shaded/hbase-shaded-with-hadoop-check-invariants hbase-archetypes . U: . |
| Console output | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/1/console |
| versions | git=2.17.1 maven=2018-06-17T18:33:14Z) |
| Powered by | Apache Yetus 0.11.1 https://yetus.apache.org |
This message was automatically generated.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [hbase] Apache9 commented on issue #1265: HBASE-23077 move entirely
to spotbugs
Posted by GitBox <gi...@apache.org>.
Apache9 commented on issue #1265: HBASE-23077 move entirely to spotbugs
URL: https://github.com/apache/hbase/pull/1265#issuecomment-597183191
Saw this in YETUS-749
```
The SpotBugs plug-in requires the SPOTBUGS_HOME environment variable to be defined. It should point to the location where SpotBugs has been installed.
```
But not sure hwo to set the ENV as do not know where is the correct SPOTBUGS_HOME...
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [hbase] Apache-HBase commented on issue #1265: HBASE-23077 move
entirely to spotbugs
Posted by GitBox <gi...@apache.org>.
Apache-HBase commented on issue #1265: HBASE-23077 move entirely to spotbugs
URL: https://github.com/apache/hbase/pull/1265#issuecomment-597651915
(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/6/console in case of problems.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [hbase] Apache-HBase commented on issue #1265: HBASE-23077 move
entirely to spotbugs
Posted by GitBox <gi...@apache.org>.
Apache-HBase commented on issue #1265: HBASE-23077 move entirely to spotbugs
URL: https://github.com/apache/hbase/pull/1265#issuecomment-597871440
(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/7/console in case of problems.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [hbase] Apache-HBase commented on issue #1265: HBASE-23077 move
entirely to spotbugs
Posted by GitBox <gi...@apache.org>.
Apache-HBase commented on issue #1265: HBASE-23077 move entirely to spotbugs
URL: https://github.com/apache/hbase/pull/1265#issuecomment-597648915
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Comment |
|:----:|----------:|--------:|:--------|
| +0 :ok: | reexec | 3m 18s | Docker mode activated. |
| -0 :warning: | yetus | 0m 3s | Unprocessed flag(s): --brief-report-file --findbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck |
||| _ Prechecks _ |
||| _ master Compile Tests _ |
| +0 :ok: | mvndep | 0m 39s | Maven dependency ordering for branch |
| +1 :green_heart: | mvninstall | 6m 11s | master passed |
| +1 :green_heart: | compile | 3m 30s | master passed |
| +1 :green_heart: | shadedjars | 5m 23s | branch has no errors when building our shaded downstream artifacts. |
| +1 :green_heart: | javadoc | 8m 51s | master passed |
||| _ Patch Compile Tests _ |
| +0 :ok: | mvndep | 0m 19s | Maven dependency ordering for patch |
| +1 :green_heart: | mvninstall | 5m 52s | the patch passed |
| +1 :green_heart: | compile | 3m 24s | the patch passed |
| +1 :green_heart: | javac | 3m 24s | the patch passed |
| +1 :green_heart: | shadedjars | 5m 20s | patch has no errors when building our shaded downstream artifacts. |
| +1 :green_heart: | javadoc | 8m 47s | the patch passed |
||| _ Other Tests _ |
| -1 :x: | unit | 173m 30s | root in the patch failed. |
| | | 230m 1s | |
| Subsystem | Report/Notes |
|----------:|:-------------|
| Docker | Client=19.03.7 Server=19.03.7 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/5/artifact/yetus-jdk8-hadoop2-check/output/Dockerfile |
| GITHUB PR | https://github.com/apache/hbase/pull/1265 |
| Optional Tests | javac javadoc unit shadedjars compile |
| uname | Linux e5e77cafd49f 4.15.0-74-generic #84-Ubuntu SMP Thu Dec 19 08:06:28 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux |
| Build tool | maven |
| Personality | dev-support/hbase-personality.sh |
| git revision | master / d019fd2eed |
| Default Java | 1.8.0_232 |
| unit | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/5/artifact/yetus-jdk8-hadoop2-check/output/patch-unit-root.txt |
| Test Results | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/5/testReport/ |
| Max. process+thread count | 6148 (vs. ulimit of 10000) |
| modules | C: hbase-protocol-shaded hbase-common hbase-client hbase-zookeeper hbase-http hbase-procedure hbase-server hbase-mapreduce hbase-thrift hbase-shell hbase-rest hbase-examples hbase-shaded hbase-shaded/hbase-shaded-client hbase-shaded/hbase-shaded-client-byo-hadoop hbase-hbtop hbase-shaded/hbase-shaded-testing-util hbase-shaded/hbase-shaded-check-invariants hbase-shaded/hbase-shaded-with-hadoop-check-invariants hbase-archetypes . U: . |
| Console output | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/5/console |
| versions | git=2.17.1 maven=2018-06-17T18:33:14Z) |
| Powered by | Apache Yetus 0.11.1 https://yetus.apache.org |
This message was automatically generated.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [hbase] Apache-HBase commented on issue #1265: HBASE-23077 move
entirely to spotbugs
Posted by GitBox <gi...@apache.org>.
Apache-HBase commented on issue #1265: HBASE-23077 move entirely to spotbugs
URL: https://github.com/apache/hbase/pull/1265#issuecomment-597194564
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Comment |
|:----:|----------:|--------:|:--------|
| +0 :ok: | reexec | 0m 33s | Docker mode activated. |
| -0 :warning: | yetus | 0m 3s | Unprocessed flag(s): --findbugs-strict-precheck |
||| _ Prechecks _ |
| +1 :green_heart: | dupname | 0m 0s | No case conflicting files found. |
| +0 :ok: | shelldocs | 0m 0s | Shelldocs was not available. |
| +1 :green_heart: | @author | 0m 0s | The patch does not contain any @author tags. |
||| _ master Compile Tests _ |
| +0 :ok: | mvndep | 0m 44s | Maven dependency ordering for branch |
||| _ Patch Compile Tests _ |
| +0 :ok: | mvndep | 0m 18s | Maven dependency ordering for patch |
| +1 :green_heart: | shellcheck | 0m 0s | There were no new shellcheck issues. |
| -0 :warning: | whitespace | 0m 0s | The patch 3 line(s) with tabs. |
| +1 :green_heart: | xml | 0m 29s | The patch has no ill-formed XML file. |
| -1 :x: | hadoopcheck | 12m 6s | The patch causes 188 errors with Hadoop v3.1.2. |
||| _ Other Tests _ |
| +1 :green_heart: | asflicense | 4m 15s | The patch does not generate ASF License warnings. |
| | | 22m 12s | |
| Subsystem | Report/Notes |
|----------:|:-------------|
| Docker | Client=19.03.7 Server=19.03.7 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/1/artifact/yetus-general-check/output/Dockerfile |
| GITHUB PR | https://github.com/apache/hbase/pull/1265 |
| Optional Tests | dupname asflicense xml shellcheck shelldocs hadoopcheck |
| uname | Linux 3bef958f5cfe 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux |
| Build tool | maven |
| Personality | dev-support/hbase-personality.sh |
| git revision | master / 31484f007f |
| whitespace | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/1/artifact/yetus-general-check/output/whitespace-tabs.txt |
| hadoopcheck | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/1/artifact/yetus-general-check/output/patch-javac-3.1.2.txt |
| Max. process+thread count | 76 (vs. ulimit of 10000) |
| modules | C: hbase-protocol-shaded hbase-common hbase-client hbase-zookeeper hbase-http hbase-procedure hbase-shaded hbase-shaded/hbase-shaded-client hbase-shaded/hbase-shaded-client-byo-hadoop hbase-hbtop hbase-shaded/hbase-shaded-with-hadoop-check-invariants hbase-archetypes . hbase-examples hbase-mapreduce hbase-rest hbase-server hbase-shaded/hbase-shaded-check-invariants hbase-shaded/hbase-shaded-testing-util hbase-shell hbase-thrift U: . |
| Console output | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/1/console |
| versions | git=2.17.1 maven=2018-06-17T18:33:14Z) shellcheck=0.4.6 |
| Powered by | Apache Yetus 0.11.1 https://yetus.apache.org |
This message was automatically generated.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [hbase] ndimiduk commented on a change in pull request #1265:
HBASE-23077 move entirely to spotbugs
Posted by GitBox <gi...@apache.org>.
ndimiduk commented on a change in pull request #1265: HBASE-23077 move entirely to spotbugs
URL: https://github.com/apache/hbase/pull/1265#discussion_r391721721
##########
File path: hbase-shaded/hbase-shaded-check-invariants/pom.xml
##########
@@ -119,7 +119,8 @@
<exclude>commons-logging:*</exclude>
<!-- annotations that never change -->
<exclude>com.google.code.findbugs:*</exclude>
- <exclude>com.github.stephenc.findbugs:*</exclude>
+ <exclude>com.github.stephenc.fingbugs:*</exclude>
Review comment:
"fingbugs" and not "findbugs" ?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [hbase] Apache-HBase commented on issue #1265: HBASE-23077 move
entirely to spotbugs
Posted by GitBox <gi...@apache.org>.
Apache-HBase commented on issue #1265: HBASE-23077 move entirely to spotbugs
URL: https://github.com/apache/hbase/pull/1265#issuecomment-597545566
(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/5/console in case of problems.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [hbase] Apache-HBase commented on issue #1265: HBASE-23077 move
entirely to spotbugs
Posted by GitBox <gi...@apache.org>.
Apache-HBase commented on issue #1265: HBASE-23077 move entirely to spotbugs
URL: https://github.com/apache/hbase/pull/1265#issuecomment-597545553
(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/5/console in case of problems.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [hbase] Apache9 merged pull request #1265: HBASE-23077 move
entirely to spotbugs
Posted by GitBox <gi...@apache.org>.
Apache9 merged pull request #1265: HBASE-23077 move entirely to spotbugs
URL: https://github.com/apache/hbase/pull/1265
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [hbase] Apache-HBase commented on issue #1265: HBASE-23077 move
entirely to spotbugs
Posted by GitBox <gi...@apache.org>.
Apache-HBase commented on issue #1265: HBASE-23077 move entirely to spotbugs
URL: https://github.com/apache/hbase/pull/1265#issuecomment-597477617
:confetti_ball: **+1 overall**
| Vote | Subsystem | Runtime | Comment |
|:----:|----------:|--------:|:--------|
| +0 :ok: | reexec | 1m 39s | Docker mode activated. |
| -0 :warning: | yetus | 0m 3s | Unprocessed flag(s): --findbugs-strict-precheck |
||| _ Prechecks _ |
| +1 :green_heart: | dupname | 0m 0s | No case conflicting files found. |
| +0 :ok: | shelldocs | 0m 0s | Shelldocs was not available. |
| +0 :ok: | spotbugs | 0m 0s | spotbugs executables are not available. |
| +1 :green_heart: | @author | 0m 0s | The patch does not contain any @author tags. |
||| _ master Compile Tests _ |
| +0 :ok: | mvndep | 0m 30s | Maven dependency ordering for branch |
||| _ Patch Compile Tests _ |
| +0 :ok: | mvndep | 0m 18s | Maven dependency ordering for patch |
| +1 :green_heart: | hadolint | 0m 2s | There were no new hadolint issues. |
| +1 :green_heart: | shellcheck | 0m 1s | There were no new shellcheck issues. |
| +1 :green_heart: | whitespace | 0m 0s | The patch has no whitespace issues. |
| +1 :green_heart: | xml | 0m 27s | The patch has no ill-formed XML file. |
| +1 :green_heart: | hadoopcheck | 18m 45s | Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2. |
||| _ Other Tests _ |
| +1 :green_heart: | asflicense | 3m 34s | The patch does not generate ASF License warnings. |
| | | 34m 48s | |
| Subsystem | Report/Notes |
|----------:|:-------------|
| Docker | Client=19.03.7 Server=19.03.7 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/4/artifact/yetus-general-check/output/Dockerfile |
| GITHUB PR | https://github.com/apache/hbase/pull/1265 |
| Optional Tests | dupname asflicense hadolint shellcheck shelldocs xml spotbugs hadoopcheck |
| uname | Linux 853d1d4df001 4.15.0-74-generic #84-Ubuntu SMP Thu Dec 19 08:06:28 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux |
| Build tool | maven |
| Personality | dev-support/hbase-personality.sh |
| git revision | master / d019fd2eed |
| Max. process+thread count | 63 (vs. ulimit of 10000) |
| modules | C: hbase-protocol-shaded hbase-common hbase-client hbase-zookeeper hbase-http hbase-procedure hbase-server hbase-mapreduce hbase-thrift hbase-shell hbase-rest hbase-examples hbase-shaded hbase-shaded/hbase-shaded-client hbase-shaded/hbase-shaded-client-byo-hadoop hbase-hbtop hbase-shaded/hbase-shaded-testing-util hbase-shaded/hbase-shaded-check-invariants hbase-shaded/hbase-shaded-with-hadoop-check-invariants hbase-archetypes . U: . |
| Console output | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/4/console |
| versions | git=2.17.1 maven=2018-06-17T18:33:14Z) shellcheck=0.4.6 hadolint=1.17.5-0-g443423c |
| Powered by | Apache Yetus 0.11.1 https://yetus.apache.org |
This message was automatically generated.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [hbase] Apache-HBase commented on issue #1265: HBASE-23077 move
entirely to spotbugs
Posted by GitBox <gi...@apache.org>.
Apache-HBase commented on issue #1265: HBASE-23077 move entirely to spotbugs
URL: https://github.com/apache/hbase/pull/1265#issuecomment-597780306
:confetti_ball: **+1 overall**
| Vote | Subsystem | Runtime | Comment |
|:----:|----------:|--------:|:--------|
| +0 :ok: | reexec | 1m 42s | Docker mode activated. |
| -0 :warning: | yetus | 0m 3s | Unprocessed flag(s): --brief-report-file --findbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck |
||| _ Prechecks _ |
||| _ master Compile Tests _ |
| +0 :ok: | mvndep | 0m 31s | Maven dependency ordering for branch |
| +1 :green_heart: | mvninstall | 7m 39s | master passed |
| +1 :green_heart: | compile | 4m 28s | master passed |
| +1 :green_heart: | shadedjars | 5m 49s | branch has no errors when building our shaded downstream artifacts. |
| -0 :warning: | javadoc | 0m 17s | hbase-common in master failed. |
| -0 :warning: | javadoc | 0m 29s | hbase-client in master failed. |
| -0 :warning: | javadoc | 0m 17s | hbase-zookeeper in master failed. |
| -0 :warning: | javadoc | 0m 17s | hbase-http in master failed. |
| -0 :warning: | javadoc | 0m 17s | hbase-procedure in master failed. |
| -0 :warning: | javadoc | 0m 46s | hbase-server in master failed. |
| -0 :warning: | javadoc | 0m 22s | hbase-mapreduce in master failed. |
| -0 :warning: | javadoc | 1m 6s | hbase-thrift in master failed. |
| -0 :warning: | javadoc | 0m 28s | hbase-rest in master failed. |
| -0 :warning: | javadoc | 0m 22s | hbase-examples in master failed. |
| -0 :warning: | javadoc | 0m 17s | hbase-hbtop in master failed. |
| -0 :warning: | javadoc | 0m 16s | hbase-archetypes in master failed. |
| -0 :warning: | javadoc | 0m 19s | root in master failed. |
||| _ Patch Compile Tests _ |
| +0 :ok: | mvndep | 0m 21s | Maven dependency ordering for patch |
| +1 :green_heart: | mvninstall | 7m 8s | the patch passed |
| +1 :green_heart: | compile | 4m 27s | the patch passed |
| +1 :green_heart: | javac | 4m 27s | the patch passed |
| +1 :green_heart: | shadedjars | 5m 53s | patch has no errors when building our shaded downstream artifacts. |
| -0 :warning: | javadoc | 0m 17s | hbase-common in the patch failed. |
| -0 :warning: | javadoc | 0m 27s | hbase-client in the patch failed. |
| -0 :warning: | javadoc | 0m 17s | hbase-zookeeper in the patch failed. |
| -0 :warning: | javadoc | 0m 18s | hbase-http in the patch failed. |
| -0 :warning: | javadoc | 0m 18s | hbase-procedure in the patch failed. |
| -0 :warning: | javadoc | 0m 49s | hbase-server in the patch failed. |
| -0 :warning: | javadoc | 0m 23s | hbase-mapreduce in the patch failed. |
| -0 :warning: | javadoc | 1m 1s | hbase-thrift in the patch failed. |
| -0 :warning: | javadoc | 0m 24s | hbase-rest in the patch failed. |
| -0 :warning: | javadoc | 0m 22s | hbase-examples in the patch failed. |
| -0 :warning: | javadoc | 0m 17s | hbase-hbtop in the patch failed. |
| -0 :warning: | javadoc | 0m 21s | hbase-archetypes in the patch failed. |
| -0 :warning: | javadoc | 0m 21s | root in the patch failed. |
||| _ Other Tests _ |
| -0 :warning: | unit | 177m 8s | root in the patch failed. |
| | | 235m 57s | |
| Subsystem | Report/Notes |
|----------:|:-------------|
| Docker | Client=19.03.7 Server=19.03.7 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/6/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile |
| GITHUB PR | https://github.com/apache/hbase/pull/1265 |
| Optional Tests | javac javadoc unit shadedjars compile |
| uname | Linux 51f0f32fe784 4.15.0-74-generic #84-Ubuntu SMP Thu Dec 19 08:06:28 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux |
| Build tool | maven |
| Personality | dev-support/hbase-personality.sh |
| git revision | master / d019fd2eed |
| Default Java | 2020-01-14 |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/6/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-common.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/6/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-client.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/6/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-zookeeper.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/6/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-http.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/6/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-procedure.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/6/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-server.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/6/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-mapreduce.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/6/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-thrift.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/6/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-rest.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/6/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-examples.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/6/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-hbtop.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/6/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-archetypes.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/6/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-root.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/6/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-common.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/6/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-client.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/6/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-zookeeper.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/6/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-http.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/6/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-procedure.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/6/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-server.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/6/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-mapreduce.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/6/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-thrift.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/6/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-rest.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/6/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-examples.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/6/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-hbtop.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/6/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-archetypes.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/6/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-root.txt |
| unit | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/6/artifact/yetus-jdk11-hadoop3-check/output/patch-unit-root.txt |
| Test Results | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/6/testReport/ |
| Max. process+thread count | 5711 (vs. ulimit of 10000) |
| modules | C: hbase-protocol-shaded hbase-common hbase-client hbase-zookeeper hbase-http hbase-procedure hbase-server hbase-mapreduce hbase-thrift hbase-shell hbase-rest hbase-examples hbase-shaded hbase-shaded/hbase-shaded-client hbase-shaded/hbase-shaded-client-byo-hadoop hbase-hbtop hbase-shaded/hbase-shaded-testing-util hbase-shaded/hbase-shaded-check-invariants hbase-shaded/hbase-shaded-with-hadoop-check-invariants hbase-archetypes . U: . |
| Console output | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/6/console |
| versions | git=2.17.1 maven=2018-06-17T18:33:14Z) |
| Powered by | Apache Yetus 0.11.1 https://yetus.apache.org |
This message was automatically generated.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [hbase] Apache-HBase commented on issue #1265: HBASE-23077 move
entirely to spotbugs
Posted by GitBox <gi...@apache.org>.
Apache-HBase commented on issue #1265: HBASE-23077 move entirely to spotbugs
URL: https://github.com/apache/hbase/pull/1265#issuecomment-597869064
(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/7/console in case of problems.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [hbase] Apache-HBase commented on issue #1265: HBASE-23077 move
entirely to spotbugs
Posted by GitBox <gi...@apache.org>.
Apache-HBase commented on issue #1265: HBASE-23077 move entirely to spotbugs
URL: https://github.com/apache/hbase/pull/1265#issuecomment-597435165
:confetti_ball: **+1 overall**
| Vote | Subsystem | Runtime | Comment |
|:----:|----------:|--------:|:--------|
| +0 :ok: | reexec | 1m 44s | Docker mode activated. |
| -0 :warning: | yetus | 0m 3s | Unprocessed flag(s): --findbugs-strict-precheck |
||| _ Prechecks _ |
| +1 :green_heart: | dupname | 0m 0s | No case conflicting files found. |
| +0 :ok: | shelldocs | 0m 0s | Shelldocs was not available. |
| +0 :ok: | spotbugs | 0m 0s | spotbugs executables are not available. |
| +1 :green_heart: | @author | 0m 0s | The patch does not contain any @author tags. |
||| _ master Compile Tests _ |
| +0 :ok: | mvndep | 0m 45s | Maven dependency ordering for branch |
||| _ Patch Compile Tests _ |
| +0 :ok: | mvndep | 0m 18s | Maven dependency ordering for patch |
| +1 :green_heart: | hadolint | 0m 2s | There were no new hadolint issues. |
| +1 :green_heart: | shellcheck | 0m 1s | There were no new shellcheck issues. |
| -0 :warning: | whitespace | 0m 0s | The patch 1 line(s) with tabs. |
| +1 :green_heart: | xml | 0m 29s | The patch has no ill-formed XML file. |
| +1 :green_heart: | hadoopcheck | 16m 15s | Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2. |
||| _ Other Tests _ |
| +1 :green_heart: | asflicense | 4m 14s | The patch does not generate ASF License warnings. |
| | | 32m 40s | |
| Subsystem | Report/Notes |
|----------:|:-------------|
| Docker | Client=19.03.7 Server=19.03.7 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/3/artifact/yetus-general-check/output/Dockerfile |
| GITHUB PR | https://github.com/apache/hbase/pull/1265 |
| Optional Tests | dupname asflicense hadolint shellcheck shelldocs xml spotbugs hadoopcheck |
| uname | Linux 043a2db9c0bd 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux |
| Build tool | maven |
| Personality | dev-support/hbase-personality.sh |
| git revision | master / e345a95461 |
| whitespace | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/3/artifact/yetus-general-check/output/whitespace-tabs.txt |
| Max. process+thread count | 76 (vs. ulimit of 10000) |
| modules | C: hbase-protocol-shaded hbase-common hbase-client hbase-zookeeper hbase-http hbase-procedure hbase-shaded hbase-shaded/hbase-shaded-client hbase-shaded/hbase-shaded-client-byo-hadoop hbase-hbtop hbase-shaded/hbase-shaded-with-hadoop-check-invariants hbase-archetypes . hbase-examples hbase-mapreduce hbase-rest hbase-server hbase-shaded/hbase-shaded-check-invariants hbase-shaded/hbase-shaded-testing-util hbase-shell hbase-thrift U: . |
| Console output | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/3/console |
| versions | git=2.17.1 maven=2018-06-17T18:33:14Z) shellcheck=0.4.6 hadolint=1.17.5-0-g443423c |
| Powered by | Apache Yetus 0.11.1 https://yetus.apache.org |
This message was automatically generated.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [hbase] Apache-HBase commented on issue #1265: HBASE-23077 move
entirely to spotbugs
Posted by GitBox <gi...@apache.org>.
Apache-HBase commented on issue #1265: HBASE-23077 move entirely to spotbugs
URL: https://github.com/apache/hbase/pull/1265#issuecomment-597997002
:confetti_ball: **+1 overall**
| Vote | Subsystem | Runtime | Comment |
|:----:|----------:|--------:|:--------|
| +0 :ok: | reexec | 3m 4s | Docker mode activated. |
| -0 :warning: | yetus | 0m 3s | Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck |
||| _ Prechecks _ |
||| _ master Compile Tests _ |
| +0 :ok: | mvndep | 0m 41s | Maven dependency ordering for branch |
| +1 :green_heart: | mvninstall | 7m 29s | master passed |
| +1 :green_heart: | compile | 4m 33s | master passed |
| +1 :green_heart: | shadedjars | 5m 37s | branch has no errors when building our shaded downstream artifacts. |
| -0 :warning: | javadoc | 0m 16s | hbase-common in master failed. |
| -0 :warning: | javadoc | 0m 26s | hbase-client in master failed. |
| -0 :warning: | javadoc | 0m 16s | hbase-zookeeper in master failed. |
| -0 :warning: | javadoc | 0m 17s | hbase-http in master failed. |
| -0 :warning: | javadoc | 0m 15s | hbase-procedure in master failed. |
| -0 :warning: | javadoc | 0m 18s | hbase-hbtop in master failed. |
| -0 :warning: | javadoc | 0m 19s | hbase-archetypes in master failed. |
| -0 :warning: | javadoc | 0m 21s | root in master failed. |
| -0 :warning: | javadoc | 0m 23s | hbase-examples in master failed. |
| -0 :warning: | javadoc | 0m 22s | hbase-mapreduce in master failed. |
| -0 :warning: | javadoc | 0m 25s | hbase-rest in master failed. |
| -0 :warning: | javadoc | 0m 47s | hbase-server in master failed. |
| -0 :warning: | javadoc | 1m 3s | hbase-thrift in master failed. |
||| _ Patch Compile Tests _ |
| +0 :ok: | mvndep | 0m 22s | Maven dependency ordering for patch |
| +1 :green_heart: | mvninstall | 10m 16s | the patch passed |
| +1 :green_heart: | compile | 6m 23s | the patch passed |
| +1 :green_heart: | javac | 6m 23s | the patch passed |
| +1 :green_heart: | shadedjars | 8m 7s | patch has no errors when building our shaded downstream artifacts. |
| -0 :warning: | javadoc | 0m 26s | hbase-common in the patch failed. |
| -0 :warning: | javadoc | 0m 38s | hbase-client in the patch failed. |
| -0 :warning: | javadoc | 0m 21s | hbase-zookeeper in the patch failed. |
| -0 :warning: | javadoc | 0m 22s | hbase-http in the patch failed. |
| -0 :warning: | javadoc | 0m 25s | hbase-procedure in the patch failed. |
| -0 :warning: | javadoc | 1m 1s | hbase-server in the patch failed. |
| -0 :warning: | javadoc | 0m 32s | hbase-mapreduce in the patch failed. |
| -0 :warning: | javadoc | 1m 29s | hbase-thrift in the patch failed. |
| -0 :warning: | javadoc | 0m 37s | hbase-rest in the patch failed. |
| -0 :warning: | javadoc | 0m 31s | hbase-examples in the patch failed. |
| -0 :warning: | javadoc | 0m 23s | hbase-hbtop in the patch failed. |
| -0 :warning: | javadoc | 0m 20s | hbase-archetypes in the patch failed. |
| -0 :warning: | javadoc | 0m 30s | root in the patch failed. |
||| _ Other Tests _ |
| -0 :warning: | unit | 152m 58s | root in the patch failed. |
| | | 222m 2s | |
| Subsystem | Report/Notes |
|----------:|:-------------|
| Docker | Client=19.03.8 Server=19.03.8 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/8/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile |
| GITHUB PR | https://github.com/apache/hbase/pull/1265 |
| Optional Tests | javac javadoc unit shadedjars compile |
| uname | Linux 2939658f2426 4.15.0-74-generic #84-Ubuntu SMP Thu Dec 19 08:06:28 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux |
| Build tool | maven |
| Personality | dev-support/hbase-personality.sh |
| git revision | master / cf121227aa |
| Default Java | 2020-01-14 |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/8/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-common.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/8/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-client.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/8/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-zookeeper.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/8/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-http.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/8/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-procedure.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/8/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-hbtop.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/8/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-archetypes.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/8/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-root.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/8/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-examples.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/8/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-mapreduce.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/8/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-rest.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/8/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-server.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/8/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-thrift.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/8/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-common.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/8/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-client.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/8/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-zookeeper.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/8/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-http.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/8/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-procedure.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/8/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-server.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/8/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-mapreduce.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/8/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-thrift.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/8/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-rest.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/8/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-examples.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/8/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-hbtop.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/8/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-archetypes.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/8/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-root.txt |
| unit | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/8/artifact/yetus-jdk11-hadoop3-check/output/patch-unit-root.txt |
| Test Results | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/8/testReport/ |
| Max. process+thread count | 3702 (vs. ulimit of 10000) |
| modules | C: hbase-protocol-shaded hbase-common hbase-client hbase-zookeeper hbase-http hbase-procedure hbase-server hbase-mapreduce hbase-thrift hbase-shell hbase-rest hbase-examples hbase-shaded hbase-shaded/hbase-shaded-client hbase-shaded/hbase-shaded-client-byo-hadoop hbase-hbtop hbase-shaded/hbase-shaded-testing-util hbase-shaded/hbase-shaded-check-invariants hbase-shaded/hbase-shaded-with-hadoop-check-invariants hbase-archetypes . U: . |
| Console output | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/8/console |
| versions | git=2.17.1 maven=2018-06-17T18:33:14Z) |
| Powered by | Apache Yetus 0.11.1 https://yetus.apache.org |
This message was automatically generated.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [hbase] Apache-HBase commented on issue #1265: HBASE-23077 move
entirely to spotbugs
Posted by GitBox <gi...@apache.org>.
Apache-HBase commented on issue #1265: HBASE-23077 move entirely to spotbugs
URL: https://github.com/apache/hbase/pull/1265#issuecomment-597651872
(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/6/console in case of problems.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [hbase] busbey commented on issue #1265: HBASE-23077 move entirely
to spotbugs
Posted by GitBox <gi...@apache.org>.
busbey commented on issue #1265: HBASE-23077 move entirely to spotbugs
URL: https://github.com/apache/hbase/pull/1265#issuecomment-597186742
yeah we would need to install SPOTBUGS in the dockerfile and then point to where we install it.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [hbase] Apache-HBase commented on issue #1265: HBASE-23077 move
entirely to spotbugs
Posted by GitBox <gi...@apache.org>.
Apache-HBase commented on issue #1265: HBASE-23077 move entirely to spotbugs
URL: https://github.com/apache/hbase/pull/1265#issuecomment-597545555
(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/5/console in case of problems.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [hbase] Apache-HBase commented on issue #1265: HBASE-23077 move
entirely to spotbugs
Posted by GitBox <gi...@apache.org>.
Apache-HBase commented on issue #1265: HBASE-23077 move entirely to spotbugs
URL: https://github.com/apache/hbase/pull/1265#issuecomment-597394075
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Comment |
|:----:|----------:|--------:|:--------|
| +0 :ok: | reexec | 0m 0s | Docker mode activated. |
| -1 :x: | docker | 0m 18s | Docker failed to build yetus/hbase:9c1f2dd0ca. |
| Subsystem | Report/Notes |
|----------:|:-------------|
| GITHUB PR | https://github.com/apache/hbase/pull/1265 |
| Console output | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/2/console |
| versions | git=2.17.1 |
| Powered by | Apache Yetus 0.11.1 https://yetus.apache.org |
This message was automatically generated.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [hbase] Apache9 commented on issue #1265: HBASE-23077 move entirely
to spotbugs
Posted by GitBox <gi...@apache.org>.
Apache9 commented on issue #1265: HBASE-23077 move entirely to spotbugs
URL: https://github.com/apache/hbase/pull/1265#issuecomment-597479659
Oh, `spotbugs executables are not available.`
Let me check what's going on here...
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [hbase] Apache9 commented on a change in pull request #1265:
HBASE-23077 move entirely to spotbugs
Posted by GitBox <gi...@apache.org>.
Apache9 commented on a change in pull request #1265: HBASE-23077 move entirely to spotbugs
URL: https://github.com/apache/hbase/pull/1265#discussion_r391369449
##########
File path: dev-support/jenkins_precommit_github_yetus.sh
##########
@@ -104,14 +104,14 @@ YETUS_ARGS+=("--reapermode=kill")
# with other jobs on systemd-enabled machines
YETUS_ARGS+=("--proclimit=10000")
YETUS_ARGS+=("--dockermemlimit=20g")
-# -1 findbugs issues that show up prior to the patch being applied
-YETUS_ARGS+=("--findbugs-strict-precheck")
+# -1 spotbugs issues that show up prior to the patch being applied
+YETUS_ARGS+=("--spotbugs-strict-precheck")
# rsync these files back into the archive dir
YETUS_ARGS+=("--archive-list=${ARCHIVE_PATTERN_LIST}")
# URL for user-side presentation in reports and such to our artifacts
YETUS_ARGS+=("--build-url-artifacts=${BUILD_URL_ARTIFACTS}")
# plugins to enable
-YETUS_ARGS+=("--plugins=${PLUGINS}")
+YETUS_ARGS+=("--plugins=${PLUGINS},-findbugs")
Review comment:
This is the design of yetus... If you want to use spotbugs then specifiy -finbugs, if you want to use findbugs then specity -spotbugs...
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [hbase] saintstack commented on a change in pull request #1265:
HBASE-23077 move entirely to spotbugs
Posted by GitBox <gi...@apache.org>.
saintstack commented on a change in pull request #1265: HBASE-23077 move entirely to spotbugs
URL: https://github.com/apache/hbase/pull/1265#discussion_r391367898
##########
File path: dev-support/hbase-personality.sh
##########
@@ -184,18 +184,30 @@ function personality_modules
done
fi
- if [[ ${testtype} == findbugs ]]; then
- # Run findbugs on each module individually to diff pre-patch and post-patch results and
+ if [[ ${testtype} == spotbugs ]]; then
+ # Run spotbugs on each module individually to diff pre-patch and post-patch results and
# report new warnings for changed modules only.
- # For some reason, findbugs on root is not working, but running on individual modules is
+ # For some reason, spotbugs on root is not working, but running on individual modules is
# working. For time being, let it run on original list of CHANGED_MODULES. HBASE-19491
for module in "${CHANGED_MODULES[@]}"; do
- # skip findbugs on hbase-shell and hbase-it. hbase-it has nothing
- # in src/main/java where findbugs goes to look
+ # skip spotbugs on hbase-shell and hbase-it. hbase-it has nothing
+ # in src/main/java where spotbugs goes to look
+ # skip hbase-shaded* as there is no java code in them
+ # skip all modules with no java code or at least, non test java code
if [[ ${module} == hbase-shell ]]; then
continue
elif [[ ${module} == hbase-it ]]; then
continue
+ elif [[ ${module} == hbase-shaded* ]]; then
Review comment:
Good
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [hbase] Apache-HBase commented on issue #1265: HBASE-23077 move
entirely to spotbugs
Posted by GitBox <gi...@apache.org>.
Apache-HBase commented on issue #1265: HBASE-23077 move entirely to spotbugs
URL: https://github.com/apache/hbase/pull/1265#issuecomment-597946980
(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/8/console in case of problems.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [hbase] Apache-HBase commented on issue #1265: HBASE-23077 move
entirely to spotbugs
Posted by GitBox <gi...@apache.org>.
Apache-HBase commented on issue #1265: HBASE-23077 move entirely to spotbugs
URL: https://github.com/apache/hbase/pull/1265#issuecomment-597869042
(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/7/console in case of problems.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [hbase] Apache9 commented on issue #1265: HBASE-23077 move entirely
to spotbugs
Posted by GitBox <gi...@apache.org>.
Apache9 commented on issue #1265: HBASE-23077 move entirely to spotbugs
URL: https://github.com/apache/hbase/pull/1265#issuecomment-597710515
OK the failed spotbugs run are all for master branch, this is reasonable as we do not declare spotbugs-maven-plugin on master before this patch here...
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [hbase] Apache9 commented on a change in pull request #1265:
HBASE-23077 move entirely to spotbugs
Posted by GitBox <gi...@apache.org>.
Apache9 commented on a change in pull request #1265: HBASE-23077 move entirely to spotbugs
URL: https://github.com/apache/hbase/pull/1265#discussion_r391971731
##########
File path: hbase-shaded/hbase-shaded-testing-util/pom.xml
##########
@@ -184,7 +184,8 @@
<exclude>org.apache.hbase:hbase-resource-bundle</exclude>
<exclude>org.slf4j:*</exclude>
<exclude>com.google.code.findbugs:*</exclude>
- <exclude>com.github.stephenc.findbugs:*</exclude>
+ <exclude>com.github.stephenc.fingbugs:*</exclude>
Review comment:
Let me do a sed -i...
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [hbase] Apache-HBase commented on issue #1265: HBASE-23077 move
entirely to spotbugs
Posted by GitBox <gi...@apache.org>.
Apache-HBase commented on issue #1265: HBASE-23077 move entirely to spotbugs
URL: https://github.com/apache/hbase/pull/1265#issuecomment-597466100
:confetti_ball: **+1 overall**
| Vote | Subsystem | Runtime | Comment |
|:----:|----------:|--------:|:--------|
| +0 :ok: | reexec | 3m 25s | Docker mode activated. |
| -0 :warning: | yetus | 0m 3s | Unprocessed flag(s): --brief-report-file --findbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck |
||| _ Prechecks _ |
||| _ master Compile Tests _ |
| +0 :ok: | mvndep | 0m 45s | Maven dependency ordering for branch |
| +1 :green_heart: | mvninstall | 7m 18s | master passed |
| +1 :green_heart: | compile | 4m 26s | master passed |
| +1 :green_heart: | shadedjars | 5m 42s | branch has no errors when building our shaded downstream artifacts. |
| -0 :warning: | javadoc | 0m 18s | hbase-common in master failed. |
| -0 :warning: | javadoc | 0m 29s | hbase-client in master failed. |
| -0 :warning: | javadoc | 0m 18s | hbase-zookeeper in master failed. |
| -0 :warning: | javadoc | 0m 17s | hbase-http in master failed. |
| -0 :warning: | javadoc | 0m 19s | hbase-procedure in master failed. |
| -0 :warning: | javadoc | 0m 16s | hbase-hbtop in master failed. |
| -0 :warning: | javadoc | 0m 17s | hbase-archetypes in master failed. |
| -0 :warning: | javadoc | 0m 19s | root in master failed. |
| -0 :warning: | javadoc | 0m 22s | hbase-examples in master failed. |
| -0 :warning: | javadoc | 0m 22s | hbase-mapreduce in master failed. |
| -0 :warning: | javadoc | 0m 26s | hbase-rest in master failed. |
| -0 :warning: | javadoc | 0m 45s | hbase-server in master failed. |
| -0 :warning: | javadoc | 0m 59s | hbase-thrift in master failed. |
||| _ Patch Compile Tests _ |
| +0 :ok: | mvndep | 0m 23s | Maven dependency ordering for patch |
| +1 :green_heart: | mvninstall | 7m 7s | the patch passed |
| +1 :green_heart: | compile | 4m 25s | the patch passed |
| +1 :green_heart: | javac | 4m 25s | the patch passed |
| +1 :green_heart: | shadedjars | 5m 47s | patch has no errors when building our shaded downstream artifacts. |
| -0 :warning: | javadoc | 0m 17s | hbase-common in the patch failed. |
| -0 :warning: | javadoc | 0m 28s | hbase-client in the patch failed. |
| -0 :warning: | javadoc | 0m 16s | hbase-zookeeper in the patch failed. |
| -0 :warning: | javadoc | 0m 17s | hbase-http in the patch failed. |
| -0 :warning: | javadoc | 0m 18s | hbase-procedure in the patch failed. |
| -0 :warning: | javadoc | 0m 48s | hbase-server in the patch failed. |
| -0 :warning: | javadoc | 0m 22s | hbase-mapreduce in the patch failed. |
| -0 :warning: | javadoc | 1m 2s | hbase-thrift in the patch failed. |
| -0 :warning: | javadoc | 0m 26s | hbase-rest in the patch failed. |
| -0 :warning: | javadoc | 0m 21s | hbase-examples in the patch failed. |
| -0 :warning: | javadoc | 0m 17s | hbase-hbtop in the patch failed. |
| -0 :warning: | javadoc | 0m 14s | hbase-archetypes in the patch failed. |
| -0 :warning: | javadoc | 0m 18s | root in the patch failed. |
||| _ Other Tests _ |
| -0 :warning: | unit | 111m 4s | root in the patch failed. |
| | | 169m 42s | |
| Subsystem | Report/Notes |
|----------:|:-------------|
| Docker | Client=19.03.7 Server=19.03.7 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/3/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile |
| GITHUB PR | https://github.com/apache/hbase/pull/1265 |
| Optional Tests | javac javadoc unit shadedjars compile |
| uname | Linux 88eabde9a335 4.15.0-74-generic #84-Ubuntu SMP Thu Dec 19 08:06:28 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux |
| Build tool | maven |
| Personality | dev-support/hbase-personality.sh |
| git revision | master / e345a95461 |
| Default Java | 2020-01-14 |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/3/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-common.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/3/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-client.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/3/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-zookeeper.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/3/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-http.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/3/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-procedure.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/3/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-hbtop.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/3/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-archetypes.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/3/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-root.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/3/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-examples.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/3/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-mapreduce.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/3/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-rest.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/3/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-server.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/3/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-thrift.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/3/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-common.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/3/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-client.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/3/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-zookeeper.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/3/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-http.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/3/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-procedure.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/3/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-server.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/3/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-mapreduce.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/3/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-thrift.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/3/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-rest.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/3/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-examples.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/3/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-hbtop.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/3/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-archetypes.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/3/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-root.txt |
| unit | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/3/artifact/yetus-jdk11-hadoop3-check/output/patch-unit-root.txt |
| Test Results | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/3/testReport/ |
| Max. process+thread count | 6608 (vs. ulimit of 10000) |
| modules | C: hbase-protocol-shaded hbase-common hbase-client hbase-zookeeper hbase-http hbase-procedure hbase-server hbase-mapreduce hbase-thrift hbase-shell hbase-rest hbase-examples hbase-shaded hbase-shaded/hbase-shaded-client hbase-shaded/hbase-shaded-client-byo-hadoop hbase-hbtop hbase-shaded/hbase-shaded-testing-util hbase-shaded/hbase-shaded-check-invariants hbase-shaded/hbase-shaded-with-hadoop-check-invariants hbase-archetypes . U: . |
| Console output | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/3/console |
| versions | git=2.17.1 maven=2018-06-17T18:33:14Z) |
| Powered by | Apache Yetus 0.11.1 https://yetus.apache.org |
This message was automatically generated.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [hbase] Apache-HBase commented on issue #1265: HBASE-23077 move
entirely to spotbugs
Posted by GitBox <gi...@apache.org>.
Apache-HBase commented on issue #1265: HBASE-23077 move entirely to spotbugs
URL: https://github.com/apache/hbase/pull/1265#issuecomment-597947015
(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/8/console in case of problems.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [hbase] saintstack commented on a change in pull request #1265:
HBASE-23077 move entirely to spotbugs
Posted by GitBox <gi...@apache.org>.
saintstack commented on a change in pull request #1265: HBASE-23077 move entirely to spotbugs
URL: https://github.com/apache/hbase/pull/1265#discussion_r391367793
##########
File path: dev-support/docker/Dockerfile
##########
@@ -184,9 +183,9 @@ RUN mkdir -p /usr/lib/jvm && \
# these values to be specified here; the various --foo-path flags do not
# propigate as expected, while these are honored.
# TODO (nd): is this really true? investigate and file a ticket.
-ENV FINDBUGS_HOME '/opt/findbugs'
+ENV SPOTBUGS_HOME '/opt/spotbugs'
ENV MAVEN_HOME '/opt/maven'
-ENV MAVEN_OPTS '-Xms6G -Xmx6G'
+ENV MAVEN_OPTS '-Xms4G -Xmx4G'
Review comment:
Good
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [hbase] Apache-HBase commented on issue #1265: HBASE-23077 move
entirely to spotbugs
Posted by GitBox <gi...@apache.org>.
Apache-HBase commented on issue #1265: HBASE-23077 move entirely to spotbugs
URL: https://github.com/apache/hbase/pull/1265#issuecomment-597650085
:confetti_ball: **+1 overall**
| Vote | Subsystem | Runtime | Comment |
|:----:|----------:|--------:|:--------|
| +0 :ok: | reexec | 3m 16s | Docker mode activated. |
| -0 :warning: | yetus | 0m 3s | Unprocessed flag(s): --brief-report-file --findbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck |
||| _ Prechecks _ |
||| _ master Compile Tests _ |
| +0 :ok: | mvndep | 0m 41s | Maven dependency ordering for branch |
| +1 :green_heart: | mvninstall | 7m 35s | master passed |
| +1 :green_heart: | compile | 4m 29s | master passed |
| +1 :green_heart: | shadedjars | 5m 55s | branch has no errors when building our shaded downstream artifacts. |
| -0 :warning: | javadoc | 0m 18s | hbase-common in master failed. |
| -0 :warning: | javadoc | 0m 29s | hbase-client in master failed. |
| -0 :warning: | javadoc | 0m 17s | hbase-zookeeper in master failed. |
| -0 :warning: | javadoc | 0m 17s | hbase-http in master failed. |
| -0 :warning: | javadoc | 0m 18s | hbase-procedure in master failed. |
| -0 :warning: | javadoc | 0m 46s | hbase-server in master failed. |
| -0 :warning: | javadoc | 0m 21s | hbase-mapreduce in master failed. |
| -0 :warning: | javadoc | 1m 3s | hbase-thrift in master failed. |
| -0 :warning: | javadoc | 0m 24s | hbase-rest in master failed. |
| -0 :warning: | javadoc | 0m 23s | hbase-examples in master failed. |
| -0 :warning: | javadoc | 0m 17s | hbase-hbtop in master failed. |
| -0 :warning: | javadoc | 0m 16s | hbase-archetypes in master failed. |
| -0 :warning: | javadoc | 0m 19s | root in master failed. |
||| _ Patch Compile Tests _ |
| +0 :ok: | mvndep | 0m 19s | Maven dependency ordering for patch |
| +1 :green_heart: | mvninstall | 7m 15s | the patch passed |
| +1 :green_heart: | compile | 4m 25s | the patch passed |
| +1 :green_heart: | javac | 4m 25s | the patch passed |
| +1 :green_heart: | shadedjars | 5m 49s | patch has no errors when building our shaded downstream artifacts. |
| -0 :warning: | javadoc | 0m 18s | hbase-common in the patch failed. |
| -0 :warning: | javadoc | 0m 27s | hbase-client in the patch failed. |
| -0 :warning: | javadoc | 0m 18s | hbase-zookeeper in the patch failed. |
| -0 :warning: | javadoc | 0m 19s | hbase-http in the patch failed. |
| -0 :warning: | javadoc | 0m 18s | hbase-procedure in the patch failed. |
| -0 :warning: | javadoc | 0m 47s | hbase-server in the patch failed. |
| -0 :warning: | javadoc | 0m 22s | hbase-mapreduce in the patch failed. |
| -0 :warning: | javadoc | 1m 3s | hbase-thrift in the patch failed. |
| -0 :warning: | javadoc | 0m 24s | hbase-rest in the patch failed. |
| -0 :warning: | javadoc | 0m 23s | hbase-examples in the patch failed. |
| -0 :warning: | javadoc | 0m 18s | hbase-hbtop in the patch failed. |
| -0 :warning: | javadoc | 0m 16s | hbase-archetypes in the patch failed. |
| -0 :warning: | javadoc | 0m 29s | root in the patch failed. |
||| _ Other Tests _ |
| -0 :warning: | unit | 172m 44s | root in the patch failed. |
| | | 232m 17s | |
| Subsystem | Report/Notes |
|----------:|:-------------|
| Docker | Client=19.03.7 Server=19.03.7 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/5/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile |
| GITHUB PR | https://github.com/apache/hbase/pull/1265 |
| Optional Tests | javac javadoc unit shadedjars compile |
| uname | Linux f7e9a60fd939 4.15.0-74-generic #84-Ubuntu SMP Thu Dec 19 08:06:28 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux |
| Build tool | maven |
| Personality | dev-support/hbase-personality.sh |
| git revision | master / d019fd2eed |
| Default Java | 2020-01-14 |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/5/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-common.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/5/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-client.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/5/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-zookeeper.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/5/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-http.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/5/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-procedure.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/5/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-server.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/5/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-mapreduce.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/5/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-thrift.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/5/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-rest.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/5/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-examples.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/5/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-hbtop.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/5/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-archetypes.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/5/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-root.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/5/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-common.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/5/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-client.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/5/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-zookeeper.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/5/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-http.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/5/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-procedure.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/5/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-server.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/5/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-mapreduce.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/5/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-thrift.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/5/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-rest.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/5/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-examples.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/5/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-hbtop.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/5/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-archetypes.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/5/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-root.txt |
| unit | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/5/artifact/yetus-jdk11-hadoop3-check/output/patch-unit-root.txt |
| Test Results | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/5/testReport/ |
| Max. process+thread count | 6246 (vs. ulimit of 10000) |
| modules | C: hbase-protocol-shaded hbase-common hbase-client hbase-zookeeper hbase-http hbase-procedure hbase-server hbase-mapreduce hbase-thrift hbase-shell hbase-rest hbase-examples hbase-shaded hbase-shaded/hbase-shaded-client hbase-shaded/hbase-shaded-client-byo-hadoop hbase-hbtop hbase-shaded/hbase-shaded-testing-util hbase-shaded/hbase-shaded-check-invariants hbase-shaded/hbase-shaded-with-hadoop-check-invariants hbase-archetypes . U: . |
| Console output | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/5/console |
| versions | git=2.17.1 maven=2018-06-17T18:33:14Z) |
| Powered by | Apache Yetus 0.11.1 https://yetus.apache.org |
This message was automatically generated.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [hbase] Apache9 commented on a change in pull request #1265:
HBASE-23077 move entirely to spotbugs
Posted by GitBox <gi...@apache.org>.
Apache9 commented on a change in pull request #1265: HBASE-23077 move entirely to spotbugs
URL: https://github.com/apache/hbase/pull/1265#discussion_r391369594
##########
File path: hbase-examples/pom.xml
##########
@@ -159,6 +159,8 @@
<dependency>
<groupId>com.github.stephenc.findbugs</groupId>
Review comment:
For hbase-client and another module, the problem is that we declare this dependency only under the hadoop 2 profile and it will cause compile error with hadoop 3. So just move it to the general dependencies section.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [hbase] Apache-HBase commented on issue #1265: HBASE-23077 move
entirely to spotbugs
Posted by GitBox <gi...@apache.org>.
Apache-HBase commented on issue #1265: HBASE-23077 move entirely to spotbugs
URL: https://github.com/apache/hbase/pull/1265#issuecomment-597467550
(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/4/console in case of problems.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [hbase] Apache-HBase commented on issue #1265: HBASE-23077 move
entirely to spotbugs
Posted by GitBox <gi...@apache.org>.
Apache-HBase commented on issue #1265: HBASE-23077 move entirely to spotbugs
URL: https://github.com/apache/hbase/pull/1265#issuecomment-597210903
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Comment |
|:----:|----------:|--------:|:--------|
| +0 :ok: | reexec | 1m 30s | Docker mode activated. |
| -0 :warning: | yetus | 0m 3s | Unprocessed flag(s): --brief-report-file --findbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck |
||| _ Prechecks _ |
||| _ master Compile Tests _ |
| +0 :ok: | mvndep | 0m 47s | Maven dependency ordering for branch |
| +1 :green_heart: | mvninstall | 7m 29s | master passed |
| +1 :green_heart: | compile | 4m 29s | master passed |
| +1 :green_heart: | shadedjars | 5m 46s | branch has no errors when building our shaded downstream artifacts. |
| -0 :warning: | javadoc | 0m 17s | hbase-common in master failed. |
| -0 :warning: | javadoc | 0m 29s | hbase-client in master failed. |
| -0 :warning: | javadoc | 0m 17s | hbase-zookeeper in master failed. |
| -0 :warning: | javadoc | 0m 17s | hbase-http in master failed. |
| -0 :warning: | javadoc | 0m 19s | hbase-procedure in master failed. |
| -0 :warning: | javadoc | 0m 18s | hbase-hbtop in master failed. |
| -0 :warning: | javadoc | 0m 17s | hbase-archetypes in master failed. |
| -0 :warning: | javadoc | 0m 22s | root in master failed. |
| -0 :warning: | javadoc | 0m 24s | hbase-examples in master failed. |
| -0 :warning: | javadoc | 0m 23s | hbase-mapreduce in master failed. |
| -0 :warning: | javadoc | 0m 25s | hbase-rest in master failed. |
| -0 :warning: | javadoc | 0m 48s | hbase-server in master failed. |
| -0 :warning: | javadoc | 0m 58s | hbase-thrift in master failed. |
||| _ Patch Compile Tests _ |
| +0 :ok: | mvndep | 0m 21s | Maven dependency ordering for patch |
| -1 :x: | mvninstall | 2m 8s | root in the patch failed. |
| -1 :x: | compile | 1m 42s | root in the patch failed. |
| -0 :warning: | javac | 1m 42s | root in the patch failed. |
| +1 :green_heart: | shadedjars | 5m 38s | patch has no errors when building our shaded downstream artifacts. |
| -0 :warning: | javadoc | 0m 16s | hbase-common in the patch failed. |
| -0 :warning: | javadoc | 0m 17s | hbase-client in the patch failed. |
| -0 :warning: | javadoc | 0m 15s | hbase-zookeeper in the patch failed. |
| -0 :warning: | javadoc | 0m 15s | hbase-http in the patch failed. |
| -0 :warning: | javadoc | 0m 17s | hbase-procedure in the patch failed. |
| -0 :warning: | javadoc | 0m 43s | hbase-server in the patch failed. |
| -0 :warning: | javadoc | 0m 15s | hbase-mapreduce in the patch failed. |
| -0 :warning: | javadoc | 0m 55s | hbase-thrift in the patch failed. |
| -0 :warning: | javadoc | 0m 24s | hbase-rest in the patch failed. |
| -0 :warning: | javadoc | 0m 21s | hbase-examples in the patch failed. |
| -0 :warning: | javadoc | 0m 18s | hbase-hbtop in the patch failed. |
| -0 :warning: | javadoc | 0m 15s | hbase-archetypes in the patch failed. |
| -0 :warning: | javadoc | 0m 21s | root in the patch failed. |
||| _ Other Tests _ |
| -0 :warning: | unit | 5m 19s | root in the patch failed. |
| | | 53m 6s | |
| Subsystem | Report/Notes |
|----------:|:-------------|
| Docker | Client=19.03.7 Server=19.03.7 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile |
| GITHUB PR | https://github.com/apache/hbase/pull/1265 |
| Optional Tests | javac javadoc unit shadedjars compile |
| uname | Linux d32d66ded999 4.15.0-74-generic #84-Ubuntu SMP Thu Dec 19 08:06:28 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux |
| Build tool | maven |
| Personality | dev-support/hbase-personality.sh |
| git revision | master / 31484f007f |
| Default Java | 2020-01-14 |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/1/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-common.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/1/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-client.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/1/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-zookeeper.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/1/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-http.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/1/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-procedure.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/1/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-hbtop.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/1/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-archetypes.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/1/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-root.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/1/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-examples.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/1/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-mapreduce.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/1/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-rest.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/1/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-server.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/1/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-thrift.txt |
| mvninstall | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/1/artifact/yetus-jdk11-hadoop3-check/output/patch-mvninstall-root.txt |
| compile | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/1/artifact/yetus-jdk11-hadoop3-check/output/patch-compile-root.txt |
| javac | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/1/artifact/yetus-jdk11-hadoop3-check/output/patch-compile-root.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/1/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-common.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/1/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-client.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/1/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-zookeeper.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/1/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-http.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/1/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-procedure.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/1/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-server.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/1/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-mapreduce.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/1/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-thrift.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/1/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-rest.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/1/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-examples.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/1/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-hbtop.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/1/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-archetypes.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/1/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-root.txt |
| unit | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/1/artifact/yetus-jdk11-hadoop3-check/output/patch-unit-root.txt |
| Test Results | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/1/testReport/ |
| Max. process+thread count | 356 (vs. ulimit of 10000) |
| modules | C: hbase-protocol-shaded hbase-common hbase-client hbase-zookeeper hbase-http hbase-procedure hbase-server hbase-mapreduce hbase-thrift hbase-shell hbase-rest hbase-examples hbase-shaded hbase-shaded/hbase-shaded-client hbase-shaded/hbase-shaded-client-byo-hadoop hbase-hbtop hbase-shaded/hbase-shaded-testing-util hbase-shaded/hbase-shaded-check-invariants hbase-shaded/hbase-shaded-with-hadoop-check-invariants hbase-archetypes . U: . |
| Console output | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/1/console |
| versions | git=2.17.1 maven=2018-06-17T18:33:14Z) |
| Powered by | Apache Yetus 0.11.1 https://yetus.apache.org |
This message was automatically generated.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [hbase] Apache-HBase commented on issue #1265: HBASE-23077 move
entirely to spotbugs
Posted by GitBox <gi...@apache.org>.
Apache-HBase commented on issue #1265: HBASE-23077 move entirely to spotbugs
URL: https://github.com/apache/hbase/pull/1265#issuecomment-597543994
:confetti_ball: **+1 overall**
| Vote | Subsystem | Runtime | Comment |
|:----:|----------:|--------:|:--------|
| +0 :ok: | reexec | 1m 41s | Docker mode activated. |
| -0 :warning: | yetus | 0m 3s | Unprocessed flag(s): --brief-report-file --findbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck |
||| _ Prechecks _ |
||| _ master Compile Tests _ |
| +0 :ok: | mvndep | 0m 28s | Maven dependency ordering for branch |
| +1 :green_heart: | mvninstall | 5m 55s | master passed |
| +1 :green_heart: | compile | 3m 25s | master passed |
| +1 :green_heart: | shadedjars | 5m 21s | branch has no errors when building our shaded downstream artifacts. |
| +1 :green_heart: | javadoc | 8m 26s | master passed |
||| _ Patch Compile Tests _ |
| +0 :ok: | mvndep | 0m 17s | Maven dependency ordering for patch |
| +1 :green_heart: | mvninstall | 5m 47s | the patch passed |
| +1 :green_heart: | compile | 3m 22s | the patch passed |
| +1 :green_heart: | javac | 3m 22s | the patch passed |
| +1 :green_heart: | shadedjars | 5m 11s | patch has no errors when building our shaded downstream artifacts. |
| +1 :green_heart: | javadoc | 8m 9s | the patch passed |
||| _ Other Tests _ |
| +1 :green_heart: | unit | 155m 6s | root in the patch passed. |
| | | 208m 16s | |
| Subsystem | Report/Notes |
|----------:|:-------------|
| Docker | Client=19.03.7 Server=19.03.7 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/4/artifact/yetus-jdk8-hadoop2-check/output/Dockerfile |
| GITHUB PR | https://github.com/apache/hbase/pull/1265 |
| Optional Tests | javac javadoc unit shadedjars compile |
| uname | Linux 2d5b82f193d0 4.15.0-74-generic #84-Ubuntu SMP Thu Dec 19 08:06:28 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux |
| Build tool | maven |
| Personality | dev-support/hbase-personality.sh |
| git revision | master / d019fd2eed |
| Default Java | 1.8.0_232 |
| Test Results | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/4/testReport/ |
| Max. process+thread count | 6828 (vs. ulimit of 10000) |
| modules | C: hbase-protocol-shaded hbase-common hbase-client hbase-zookeeper hbase-http hbase-procedure hbase-server hbase-mapreduce hbase-thrift hbase-shell hbase-rest hbase-examples hbase-shaded hbase-shaded/hbase-shaded-client hbase-shaded/hbase-shaded-client-byo-hadoop hbase-hbtop hbase-shaded/hbase-shaded-testing-util hbase-shaded/hbase-shaded-check-invariants hbase-shaded/hbase-shaded-with-hadoop-check-invariants hbase-archetypes . U: . |
| Console output | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/4/console |
| versions | git=2.17.1 maven=2018-06-17T18:33:14Z) |
| Powered by | Apache Yetus 0.11.1 https://yetus.apache.org |
This message was automatically generated.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [hbase] Apache9 commented on issue #1265: HBASE-23077 move entirely
to spotbugs
Posted by GitBox <gi...@apache.org>.
Apache9 commented on issue #1265: HBASE-23077 move entirely to spotbugs
URL: https://github.com/apache/hbase/pull/1265#issuecomment-597183770
Need to edit the Dockerfile?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [hbase] Apache-HBase commented on issue #1265: HBASE-23077 move
entirely to spotbugs
Posted by GitBox <gi...@apache.org>.
Apache-HBase commented on issue #1265: HBASE-23077 move entirely to spotbugs
URL: https://github.com/apache/hbase/pull/1265#issuecomment-597394005
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Comment |
|:----:|----------:|--------:|:--------|
| +0 :ok: | reexec | 0m 0s | Docker mode activated. |
| -1 :x: | docker | 0m 6s | Docker failed to build yetus/hbase:9c1f2dd0ca. |
| Subsystem | Report/Notes |
|----------:|:-------------|
| GITHUB PR | https://github.com/apache/hbase/pull/1265 |
| Console output | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/2/console |
| versions | git=2.17.1 |
| Powered by | Apache Yetus 0.11.1 https://yetus.apache.org |
This message was automatically generated.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [hbase] Apache-HBase commented on issue #1265: HBASE-23077 move
entirely to spotbugs
Posted by GitBox <gi...@apache.org>.
Apache-HBase commented on issue #1265: HBASE-23077 move entirely to spotbugs
URL: https://github.com/apache/hbase/pull/1265#issuecomment-597394076
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Comment |
|:----:|----------:|--------:|:--------|
| +0 :ok: | reexec | 0m 0s | Docker mode activated. |
| -1 :x: | docker | 0m 17s | Docker failed to build yetus/hbase:9c1f2dd0ca. |
| Subsystem | Report/Notes |
|----------:|:-------------|
| GITHUB PR | https://github.com/apache/hbase/pull/1265 |
| Console output | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/2/console |
| versions | git=2.17.1 |
| Powered by | Apache Yetus 0.11.1 https://yetus.apache.org |
This message was automatically generated.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [hbase] ndimiduk commented on a change in pull request #1265:
HBASE-23077 move entirely to spotbugs
Posted by GitBox <gi...@apache.org>.
ndimiduk commented on a change in pull request #1265: HBASE-23077 move entirely to spotbugs
URL: https://github.com/apache/hbase/pull/1265#discussion_r391721840
##########
File path: hbase-shaded/hbase-shaded-client/pom.xml
##########
@@ -67,7 +67,8 @@
<exclude>org.apache.hbase:hbase-resource-bundle</exclude>
<exclude>org.slf4j:*</exclude>
<exclude>com.google.code.findbugs:*</exclude>
- <exclude>com.github.stephenc.findbugs:*</exclude>
+ <exclude>com.github.stephenc.fingbugs:*</exclude>
Review comment:
same.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [hbase] saintstack commented on a change in pull request #1265:
HBASE-23077 move entirely to spotbugs
Posted by GitBox <gi...@apache.org>.
saintstack commented on a change in pull request #1265: HBASE-23077 move entirely to spotbugs
URL: https://github.com/apache/hbase/pull/1265#discussion_r391368354
##########
File path: hbase-examples/pom.xml
##########
@@ -159,6 +159,8 @@
<dependency>
<groupId>com.github.stephenc.findbugs</groupId>
Review comment:
Don't need to remove this stephenc findbugs? You do above in hbase-client/pom.xml You seem to leave it in elsewhere so guessing it ok.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [hbase] Apache-HBase commented on issue #1265: HBASE-23077 move
entirely to spotbugs
Posted by GitBox <gi...@apache.org>.
Apache-HBase commented on issue #1265: HBASE-23077 move entirely to spotbugs
URL: https://github.com/apache/hbase/pull/1265#issuecomment-597576430
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Comment |
|:----:|----------:|--------:|:--------|
| +0 :ok: | reexec | 1m 39s | Docker mode activated. |
| -0 :warning: | yetus | 0m 3s | Unprocessed flag(s): --findbugs-strict-precheck |
||| _ Prechecks _ |
| +1 :green_heart: | dupname | 0m 1s | No case conflicting files found. |
| +0 :ok: | shelldocs | 0m 1s | Shelldocs was not available. |
| +1 :green_heart: | @author | 0m 0s | The patch does not contain any @author tags. |
||| _ master Compile Tests _ |
| +0 :ok: | mvndep | 0m 29s | Maven dependency ordering for branch |
| +1 :green_heart: | mvninstall | 5m 8s | master passed |
| -1 :x: | spotbugs | 0m 12s | hbase-protocol-shaded in master failed. |
| -1 :x: | spotbugs | 0m 12s | hbase-common in master failed. |
| -1 :x: | spotbugs | 0m 12s | hbase-client in master failed. |
| -1 :x: | spotbugs | 0m 11s | hbase-zookeeper in master failed. |
| -1 :x: | spotbugs | 0m 12s | hbase-http in master failed. |
| -1 :x: | spotbugs | 0m 12s | hbase-procedure in master failed. |
| -1 :x: | spotbugs | 0m 12s | hbase-server in master failed. |
| -1 :x: | spotbugs | 0m 11s | hbase-mapreduce in master failed. |
| -1 :x: | spotbugs | 0m 12s | hbase-thrift in master failed. |
| -1 :x: | spotbugs | 0m 12s | hbase-rest in master failed. |
| -1 :x: | spotbugs | 0m 12s | hbase-examples in master failed. |
| -1 :x: | spotbugs | 0m 12s | hbase-shaded in master failed. |
| -1 :x: | spotbugs | 0m 13s | hbase-shaded-client in master failed. |
| -1 :x: | spotbugs | 0m 12s | hbase-shaded-client-byo-hadoop in master failed. |
| -1 :x: | spotbugs | 0m 12s | hbase-hbtop in master failed. |
| -1 :x: | spotbugs | 0m 12s | hbase-shaded-testing-util in master failed. |
| -1 :x: | spotbugs | 0m 11s | hbase-shaded-check-invariants in master failed. |
| -1 :x: | spotbugs | 0m 12s | hbase-shaded-with-hadoop-check-invariants in master failed. |
| -1 :x: | spotbugs | 0m 12s | hbase-archetypes in master failed. |
| -1 :x: | spotbugs | 0m 12s | root in master failed. |
||| _ Patch Compile Tests _ |
| +0 :ok: | mvndep | 0m 20s | Maven dependency ordering for patch |
| +1 :green_heart: | mvninstall | 5m 5s | the patch passed |
| -1 :x: | hadolint | 0m 3s | The patch generated 1 new + 0 unchanged - 0 fixed = 1 total (was 0) |
| +1 :green_heart: | shellcheck | 0m 2s | There were no new shellcheck issues. |
| +1 :green_heart: | whitespace | 0m 0s | The patch has no whitespace issues. |
| +1 :green_heart: | xml | 0m 29s | The patch has no ill-formed XML file. |
| +1 :green_heart: | hadoopcheck | 16m 1s | Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2. |
| -1 :x: | spotbugs | 2m 51s | hbase-server in the patch failed. |
| +0 :ok: | spotbugs | 0m 39s | hbase-shaded has no data from spotbugs |
| +0 :ok: | spotbugs | 0m 17s | hbase-shaded/hbase-shaded-client has no data from spotbugs |
| +0 :ok: | spotbugs | 0m 19s | hbase-shaded/hbase-shaded-client-byo-hadoop has no data from spotbugs |
| +0 :ok: | spotbugs | 0m 24s | hbase-shaded/hbase-shaded-testing-util has no data from spotbugs |
| +0 :ok: | spotbugs | 0m 14s | hbase-shaded/hbase-shaded-check-invariants has no data from spotbugs |
| +0 :ok: | spotbugs | 0m 14s | hbase-shaded/hbase-shaded-with-hadoop-check-invariants has no data from spotbugs |
| -1 :x: | spotbugs | 8m 46s | root in the patch failed. |
||| _ Other Tests _ |
| +1 :green_heart: | asflicense | 4m 12s | The patch does not generate ASF License warnings. |
| | | 73m 13s | |
| Subsystem | Report/Notes |
|----------:|:-------------|
| Docker | Client=19.03.7 Server=19.03.7 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/5/artifact/yetus-general-check/output/Dockerfile |
| GITHUB PR | https://github.com/apache/hbase/pull/1265 |
| Optional Tests | dupname asflicense hadolint shellcheck shelldocs xml spotbugs hadoopcheck |
| uname | Linux 097b32c732f2 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux |
| Build tool | maven |
| Personality | dev-support/hbase-personality.sh |
| git revision | master / d019fd2eed |
| spotbugs | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/5/artifact/yetus-general-check/output/branch-spotbugs-hbase-protocol-shaded.txt |
| spotbugs | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/5/artifact/yetus-general-check/output/branch-spotbugs-hbase-common.txt |
| spotbugs | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/5/artifact/yetus-general-check/output/branch-spotbugs-hbase-client.txt |
| spotbugs | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/5/artifact/yetus-general-check/output/branch-spotbugs-hbase-zookeeper.txt |
| spotbugs | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/5/artifact/yetus-general-check/output/branch-spotbugs-hbase-http.txt |
| spotbugs | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/5/artifact/yetus-general-check/output/branch-spotbugs-hbase-procedure.txt |
| spotbugs | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/5/artifact/yetus-general-check/output/branch-spotbugs-hbase-server.txt |
| spotbugs | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/5/artifact/yetus-general-check/output/branch-spotbugs-hbase-mapreduce.txt |
| spotbugs | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/5/artifact/yetus-general-check/output/branch-spotbugs-hbase-thrift.txt |
| spotbugs | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/5/artifact/yetus-general-check/output/branch-spotbugs-hbase-rest.txt |
| spotbugs | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/5/artifact/yetus-general-check/output/branch-spotbugs-hbase-examples.txt |
| spotbugs | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/5/artifact/yetus-general-check/output/branch-spotbugs-hbase-shaded.txt |
| spotbugs | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/5/artifact/yetus-general-check/output/branch-spotbugs-hbase-shaded_hbase-shaded-client.txt |
| spotbugs | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/5/artifact/yetus-general-check/output/branch-spotbugs-hbase-shaded_hbase-shaded-client-byo-hadoop.txt |
| spotbugs | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/5/artifact/yetus-general-check/output/branch-spotbugs-hbase-hbtop.txt |
| spotbugs | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/5/artifact/yetus-general-check/output/branch-spotbugs-hbase-shaded_hbase-shaded-testing-util.txt |
| spotbugs | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/5/artifact/yetus-general-check/output/branch-spotbugs-hbase-shaded_hbase-shaded-check-invariants.txt |
| spotbugs | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/5/artifact/yetus-general-check/output/branch-spotbugs-hbase-shaded_hbase-shaded-with-hadoop-check-invariants.txt |
| spotbugs | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/5/artifact/yetus-general-check/output/branch-spotbugs-hbase-archetypes.txt |
| spotbugs | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/5/artifact/yetus-general-check/output/branch-spotbugs-root.txt |
| hadolint | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/5/artifact/yetus-general-check/output/diff-patch-hadolint.txt |
| spotbugs | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/5/artifact/yetus-general-check/output/patch-spotbugs-hbase-server.txt |
| spotbugs | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/5/artifact/yetus-general-check/output/patch-spotbugs-root.txt |
| Max. process+thread count | 93 (vs. ulimit of 10000) |
| modules | C: hbase-protocol-shaded hbase-common hbase-client hbase-zookeeper hbase-http hbase-procedure hbase-server hbase-mapreduce hbase-thrift hbase-shell hbase-rest hbase-examples hbase-shaded hbase-shaded/hbase-shaded-client hbase-shaded/hbase-shaded-client-byo-hadoop hbase-hbtop hbase-shaded/hbase-shaded-testing-util hbase-shaded/hbase-shaded-check-invariants hbase-shaded/hbase-shaded-with-hadoop-check-invariants hbase-archetypes . U: . |
| Console output | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/5/console |
| versions | git=2.17.1 maven=2018-06-17T18:33:14Z) shellcheck=0.4.6 hadolint=1.17.5-0-g443423c |
| Powered by | Apache Yetus 0.11.1 https://yetus.apache.org |
This message was automatically generated.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [hbase] Apache-HBase commented on issue #1265: HBASE-23077 move
entirely to spotbugs
Posted by GitBox <gi...@apache.org>.
Apache-HBase commented on issue #1265: HBASE-23077 move entirely to spotbugs
URL: https://github.com/apache/hbase/pull/1265#issuecomment-597467581
(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/4/console in case of problems.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [hbase] busbey commented on a change in pull request #1265:
HBASE-23077 move entirely to spotbugs
Posted by GitBox <gi...@apache.org>.
busbey commented on a change in pull request #1265: HBASE-23077 move entirely to spotbugs
URL: https://github.com/apache/hbase/pull/1265#discussion_r390453655
##########
File path: hbase-shaded/hbase-shaded-testing-util/pom.xml
##########
@@ -183,8 +183,9 @@
<!-- The rest of these should be kept in sync with the parent pom -->
<exclude>org.apache.hbase:hbase-resource-bundle</exclude>
<exclude>org.slf4j:*</exclude>
- <exclude>com.google.code.findbugs:*</exclude>
- <exclude>com.github.stephenc.findbugs:*</exclude>
+ <exclude>com.google.code.findbugs:*</exclude>
Review comment:
nit: whitespace error
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [hbase] Apache-HBase commented on issue #1265: HBASE-23077 move
entirely to spotbugs
Posted by GitBox <gi...@apache.org>.
Apache-HBase commented on issue #1265: HBASE-23077 move entirely to spotbugs
URL: https://github.com/apache/hbase/pull/1265#issuecomment-597393985
(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/2/console in case of problems.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [hbase] Apache-HBase commented on issue #1265: HBASE-23077 move
entirely to spotbugs
Posted by GitBox <gi...@apache.org>.
Apache-HBase commented on issue #1265: HBASE-23077 move entirely to spotbugs
URL: https://github.com/apache/hbase/pull/1265#issuecomment-597946986
(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/8/console in case of problems.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [hbase] Apache-HBase commented on issue #1265: HBASE-23077 move
entirely to spotbugs
Posted by GitBox <gi...@apache.org>.
Apache-HBase commented on issue #1265: HBASE-23077 move entirely to spotbugs
URL: https://github.com/apache/hbase/pull/1265#issuecomment-597932423
:confetti_ball: **+1 overall**
| Vote | Subsystem | Runtime | Comment |
|:----:|----------:|--------:|:--------|
| +0 :ok: | reexec | 4m 6s | Docker mode activated. |
| -0 :warning: | yetus | 0m 3s | Unprocessed flag(s): --brief-report-file --findbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck |
||| _ Prechecks _ |
||| _ master Compile Tests _ |
| +0 :ok: | mvndep | 0m 43s | Maven dependency ordering for branch |
| +1 :green_heart: | mvninstall | 7m 18s | master passed |
| +1 :green_heart: | compile | 4m 28s | master passed |
| +1 :green_heart: | shadedjars | 5m 42s | branch has no errors when building our shaded downstream artifacts. |
| -0 :warning: | javadoc | 0m 16s | hbase-common in master failed. |
| -0 :warning: | javadoc | 0m 25s | hbase-client in master failed. |
| -0 :warning: | javadoc | 0m 14s | hbase-zookeeper in master failed. |
| -0 :warning: | javadoc | 0m 16s | hbase-http in master failed. |
| -0 :warning: | javadoc | 0m 16s | hbase-procedure in master failed. |
| -0 :warning: | javadoc | 0m 16s | hbase-hbtop in master failed. |
| -0 :warning: | javadoc | 0m 14s | hbase-archetypes in master failed. |
| -0 :warning: | javadoc | 0m 18s | root in master failed. |
| -0 :warning: | javadoc | 0m 19s | hbase-examples in master failed. |
| -0 :warning: | javadoc | 0m 19s | hbase-mapreduce in master failed. |
| -0 :warning: | javadoc | 0m 21s | hbase-rest in master failed. |
| -0 :warning: | javadoc | 0m 40s | hbase-server in master failed. |
| -0 :warning: | javadoc | 0m 56s | hbase-thrift in master failed. |
||| _ Patch Compile Tests _ |
| +0 :ok: | mvndep | 0m 20s | Maven dependency ordering for patch |
| +1 :green_heart: | mvninstall | 7m 1s | the patch passed |
| +1 :green_heart: | compile | 4m 19s | the patch passed |
| +1 :green_heart: | javac | 4m 19s | the patch passed |
| +1 :green_heart: | shadedjars | 5m 25s | patch has no errors when building our shaded downstream artifacts. |
| -0 :warning: | javadoc | 0m 15s | hbase-common in the patch failed. |
| -0 :warning: | javadoc | 0m 24s | hbase-client in the patch failed. |
| -0 :warning: | javadoc | 0m 14s | hbase-zookeeper in the patch failed. |
| -0 :warning: | javadoc | 0m 16s | hbase-http in the patch failed. |
| -0 :warning: | javadoc | 0m 17s | hbase-procedure in the patch failed. |
| -0 :warning: | javadoc | 0m 40s | hbase-server in the patch failed. |
| -0 :warning: | javadoc | 0m 18s | hbase-mapreduce in the patch failed. |
| -0 :warning: | javadoc | 0m 56s | hbase-thrift in the patch failed. |
| -0 :warning: | javadoc | 0m 22s | hbase-rest in the patch failed. |
| -0 :warning: | javadoc | 0m 21s | hbase-examples in the patch failed. |
| -0 :warning: | javadoc | 0m 16s | hbase-hbtop in the patch failed. |
| -0 :warning: | javadoc | 0m 14s | hbase-archetypes in the patch failed. |
| -0 :warning: | javadoc | 0m 18s | root in the patch failed. |
||| _ Other Tests _ |
| -0 :warning: | unit | 111m 42s | root in the patch failed. |
| | | 168m 32s | |
| Subsystem | Report/Notes |
|----------:|:-------------|
| Docker | Client=19.03.8 Server=19.03.8 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/7/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile |
| GITHUB PR | https://github.com/apache/hbase/pull/1265 |
| Optional Tests | javac javadoc unit shadedjars compile |
| uname | Linux 71ea0ba6fcd4 4.15.0-74-generic #84-Ubuntu SMP Thu Dec 19 08:06:28 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux |
| Build tool | maven |
| Personality | dev-support/hbase-personality.sh |
| git revision | master / aaf9a148a0 |
| Default Java | 2020-01-14 |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/7/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-common.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/7/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-client.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/7/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-zookeeper.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/7/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-http.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/7/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-procedure.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/7/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-hbtop.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/7/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-archetypes.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/7/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-root.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/7/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-examples.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/7/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-mapreduce.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/7/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-rest.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/7/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-server.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/7/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-thrift.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/7/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-common.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/7/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-client.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/7/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-zookeeper.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/7/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-http.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/7/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-procedure.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/7/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-server.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/7/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-mapreduce.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/7/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-thrift.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/7/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-rest.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/7/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-examples.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/7/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-hbtop.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/7/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-archetypes.txt |
| javadoc | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/7/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-root.txt |
| unit | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/7/artifact/yetus-jdk11-hadoop3-check/output/patch-unit-root.txt |
| Test Results | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/7/testReport/ |
| Max. process+thread count | 4086 (vs. ulimit of 10000) |
| modules | C: hbase-protocol-shaded hbase-common hbase-client hbase-zookeeper hbase-http hbase-procedure hbase-server hbase-mapreduce hbase-thrift hbase-shell hbase-rest hbase-examples hbase-shaded hbase-shaded/hbase-shaded-client hbase-shaded/hbase-shaded-client-byo-hadoop hbase-hbtop hbase-shaded/hbase-shaded-testing-util hbase-shaded/hbase-shaded-check-invariants hbase-shaded/hbase-shaded-with-hadoop-check-invariants hbase-archetypes . U: . |
| Console output | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/7/console |
| versions | git=2.17.1 maven=2018-06-17T18:33:14Z) |
| Powered by | Apache Yetus 0.11.1 https://yetus.apache.org |
This message was automatically generated.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [hbase] saintstack commented on a change in pull request #1265:
HBASE-23077 move entirely to spotbugs
Posted by GitBox <gi...@apache.org>.
saintstack commented on a change in pull request #1265: HBASE-23077 move entirely to spotbugs
URL: https://github.com/apache/hbase/pull/1265#discussion_r391368055
##########
File path: dev-support/jenkins_precommit_github_yetus.sh
##########
@@ -104,14 +104,14 @@ YETUS_ARGS+=("--reapermode=kill")
# with other jobs on systemd-enabled machines
YETUS_ARGS+=("--proclimit=10000")
YETUS_ARGS+=("--dockermemlimit=20g")
-# -1 findbugs issues that show up prior to the patch being applied
-YETUS_ARGS+=("--findbugs-strict-precheck")
+# -1 spotbugs issues that show up prior to the patch being applied
+YETUS_ARGS+=("--spotbugs-strict-precheck")
# rsync these files back into the archive dir
YETUS_ARGS+=("--archive-list=${ARCHIVE_PATTERN_LIST}")
# URL for user-side presentation in reports and such to our artifacts
YETUS_ARGS+=("--build-url-artifacts=${BUILD_URL_ARTIFACTS}")
# plugins to enable
-YETUS_ARGS+=("--plugins=${PLUGINS}")
+YETUS_ARGS+=("--plugins=${PLUGINS},-findbugs")
Review comment:
Says '--findbugs'... No '--spotbugs'?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [hbase] Apache-HBase commented on issue #1265: HBASE-23077 move
entirely to spotbugs
Posted by GitBox <gi...@apache.org>.
Apache-HBase commented on issue #1265: HBASE-23077 move entirely to spotbugs
URL: https://github.com/apache/hbase/pull/1265#issuecomment-597651914
(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/6/console in case of problems.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [hbase] Apache-HBase commented on issue #1265: HBASE-23077 move
entirely to spotbugs
Posted by GitBox <gi...@apache.org>.
Apache-HBase commented on issue #1265: HBASE-23077 move entirely to spotbugs
URL: https://github.com/apache/hbase/pull/1265#issuecomment-597899957
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Comment |
|:----:|----------:|--------:|:--------|
| +0 :ok: | reexec | 1m 53s | Docker mode activated. |
| -0 :warning: | yetus | 0m 3s | Unprocessed flag(s): --findbugs-strict-precheck |
||| _ Prechecks _ |
| +1 :green_heart: | dupname | 0m 0s | No case conflicting files found. |
| +0 :ok: | shelldocs | 0m 0s | Shelldocs was not available. |
| +1 :green_heart: | @author | 0m 0s | The patch does not contain any @author tags. |
||| _ master Compile Tests _ |
| +0 :ok: | mvndep | 0m 43s | Maven dependency ordering for branch |
| +1 :green_heart: | mvninstall | 5m 24s | master passed |
| -1 :x: | spotbugs | 0m 12s | hbase-protocol-shaded in master failed. |
| -1 :x: | spotbugs | 0m 13s | hbase-common in master failed. |
| -1 :x: | spotbugs | 0m 12s | hbase-client in master failed. |
| -1 :x: | spotbugs | 0m 12s | hbase-zookeeper in master failed. |
| -1 :x: | spotbugs | 0m 12s | hbase-http in master failed. |
| -1 :x: | spotbugs | 0m 12s | hbase-procedure in master failed. |
| -1 :x: | spotbugs | 0m 12s | hbase-hbtop in master failed. |
| -1 :x: | spotbugs | 0m 12s | hbase-archetypes in master failed. |
| -1 :x: | spotbugs | 0m 13s | root in master failed. |
| -1 :x: | spotbugs | 0m 10s | hbase-examples in master failed. |
| -1 :x: | spotbugs | 0m 12s | hbase-mapreduce in master failed. |
| -1 :x: | spotbugs | 0m 11s | hbase-rest in master failed. |
| -1 :x: | spotbugs | 0m 11s | hbase-server in master failed. |
| -1 :x: | spotbugs | 0m 10s | hbase-thrift in master failed. |
||| _ Patch Compile Tests _ |
| +0 :ok: | mvndep | 0m 23s | Maven dependency ordering for patch |
| +1 :green_heart: | mvninstall | 6m 34s | the patch passed |
| +1 :green_heart: | hadolint | 0m 2s | There were no new hadolint issues. |
| +1 :green_heart: | shellcheck | 0m 2s | There were no new shellcheck issues. |
| +1 :green_heart: | whitespace | 0m 0s | The patch has no whitespace issues. |
| +1 :green_heart: | xml | 0m 30s | The patch has no ill-formed XML file. |
| +1 :green_heart: | hadoopcheck | 18m 48s | Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2. |
| +1 :green_heart: | spotbugs | 27m 50s | the patch passed |
||| _ Other Tests _ |
| +1 :green_heart: | asflicense | 4m 0s | The patch does not generate ASF License warnings. |
| | | 80m 5s | |
| Subsystem | Report/Notes |
|----------:|:-------------|
| Docker | Client=19.03.8 Server=19.03.8 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/7/artifact/yetus-general-check/output/Dockerfile |
| GITHUB PR | https://github.com/apache/hbase/pull/1265 |
| Optional Tests | dupname asflicense hadolint shellcheck shelldocs xml spotbugs hadoopcheck |
| uname | Linux 114cf2e4ab4d 4.15.0-60-generic #67-Ubuntu SMP Thu Aug 22 16:55:30 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux |
| Build tool | maven |
| Personality | dev-support/hbase-personality.sh |
| git revision | master / aaf9a148a0 |
| spotbugs | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/7/artifact/yetus-general-check/output/branch-spotbugs-hbase-protocol-shaded.txt |
| spotbugs | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/7/artifact/yetus-general-check/output/branch-spotbugs-hbase-common.txt |
| spotbugs | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/7/artifact/yetus-general-check/output/branch-spotbugs-hbase-client.txt |
| spotbugs | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/7/artifact/yetus-general-check/output/branch-spotbugs-hbase-zookeeper.txt |
| spotbugs | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/7/artifact/yetus-general-check/output/branch-spotbugs-hbase-http.txt |
| spotbugs | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/7/artifact/yetus-general-check/output/branch-spotbugs-hbase-procedure.txt |
| spotbugs | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/7/artifact/yetus-general-check/output/branch-spotbugs-hbase-hbtop.txt |
| spotbugs | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/7/artifact/yetus-general-check/output/branch-spotbugs-hbase-archetypes.txt |
| spotbugs | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/7/artifact/yetus-general-check/output/branch-spotbugs-root.txt |
| spotbugs | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/7/artifact/yetus-general-check/output/branch-spotbugs-hbase-examples.txt |
| spotbugs | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/7/artifact/yetus-general-check/output/branch-spotbugs-hbase-mapreduce.txt |
| spotbugs | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/7/artifact/yetus-general-check/output/branch-spotbugs-hbase-rest.txt |
| spotbugs | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/7/artifact/yetus-general-check/output/branch-spotbugs-hbase-server.txt |
| spotbugs | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/7/artifact/yetus-general-check/output/branch-spotbugs-hbase-thrift.txt |
| Max. process+thread count | 93 (vs. ulimit of 10000) |
| modules | C: hbase-protocol-shaded hbase-common hbase-client hbase-zookeeper hbase-http hbase-procedure hbase-server hbase-mapreduce hbase-thrift hbase-shell hbase-rest hbase-examples hbase-shaded hbase-shaded/hbase-shaded-client hbase-shaded/hbase-shaded-client-byo-hadoop hbase-hbtop hbase-shaded/hbase-shaded-testing-util hbase-shaded/hbase-shaded-check-invariants hbase-shaded/hbase-shaded-with-hadoop-check-invariants hbase-archetypes . U: . |
| Console output | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/7/console |
| versions | git=2.17.1 maven=2018-06-17T18:33:14Z) shellcheck=0.4.6 hadolint=1.17.5-0-g443423c |
| Powered by | Apache Yetus 0.11.1 https://yetus.apache.org |
This message was automatically generated.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [hbase] Apache-HBase commented on issue #1265: HBASE-23077 move
entirely to spotbugs
Posted by GitBox <gi...@apache.org>.
Apache-HBase commented on issue #1265: HBASE-23077 move entirely to spotbugs
URL: https://github.com/apache/hbase/pull/1265#issuecomment-597690581
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Comment |
|:----:|----------:|--------:|:--------|
| +0 :ok: | reexec | 0m 32s | Docker mode activated. |
| -0 :warning: | yetus | 0m 3s | Unprocessed flag(s): --findbugs-strict-precheck |
||| _ Prechecks _ |
| +1 :green_heart: | dupname | 0m 1s | No case conflicting files found. |
| +0 :ok: | shelldocs | 0m 1s | Shelldocs was not available. |
| +1 :green_heart: | @author | 0m 0s | The patch does not contain any @author tags. |
||| _ master Compile Tests _ |
| +0 :ok: | mvndep | 0m 19s | Maven dependency ordering for branch |
| +1 :green_heart: | mvninstall | 5m 8s | master passed |
| -1 :x: | spotbugs | 0m 12s | hbase-protocol-shaded in master failed. |
| -1 :x: | spotbugs | 0m 12s | hbase-common in master failed. |
| -1 :x: | spotbugs | 0m 13s | hbase-client in master failed. |
| -1 :x: | spotbugs | 0m 12s | hbase-zookeeper in master failed. |
| -1 :x: | spotbugs | 0m 11s | hbase-http in master failed. |
| -1 :x: | spotbugs | 0m 12s | hbase-procedure in master failed. |
| -1 :x: | spotbugs | 0m 12s | hbase-server in master failed. |
| -1 :x: | spotbugs | 0m 12s | hbase-mapreduce in master failed. |
| -1 :x: | spotbugs | 0m 11s | hbase-thrift in master failed. |
| -1 :x: | spotbugs | 0m 12s | hbase-rest in master failed. |
| -1 :x: | spotbugs | 0m 12s | hbase-examples in master failed. |
| -1 :x: | spotbugs | 0m 12s | hbase-hbtop in master failed. |
| -1 :x: | spotbugs | 0m 12s | hbase-archetypes in master failed. |
| -1 :x: | spotbugs | 0m 12s | root in master failed. |
||| _ Patch Compile Tests _ |
| +0 :ok: | mvndep | 0m 20s | Maven dependency ordering for patch |
| +1 :green_heart: | mvninstall | 5m 3s | the patch passed |
| -1 :x: | hadolint | 0m 1s | The patch generated 1 new + 0 unchanged - 0 fixed = 1 total (was 0) |
| +1 :green_heart: | shellcheck | 0m 2s | There were no new shellcheck issues. |
| +1 :green_heart: | whitespace | 0m 0s | The patch has no whitespace issues. |
| +1 :green_heart: | xml | 0m 28s | The patch has no ill-formed XML file. |
| +1 :green_heart: | hadoopcheck | 15m 51s | Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2. |
| +1 :green_heart: | spotbugs | 25m 50s | the patch passed |
||| _ Other Tests _ |
| +1 :green_heart: | asflicense | 4m 13s | The patch does not generate ASF License warnings. |
| | | 70m 52s | |
| Subsystem | Report/Notes |
|----------:|:-------------|
| Docker | Client=19.03.7 Server=19.03.7 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/6/artifact/yetus-general-check/output/Dockerfile |
| GITHUB PR | https://github.com/apache/hbase/pull/1265 |
| Optional Tests | dupname asflicense hadolint shellcheck shelldocs xml spotbugs hadoopcheck |
| uname | Linux 37c5d44904e6 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux |
| Build tool | maven |
| Personality | dev-support/hbase-personality.sh |
| git revision | master / d019fd2eed |
| spotbugs | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/6/artifact/yetus-general-check/output/branch-spotbugs-hbase-protocol-shaded.txt |
| spotbugs | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/6/artifact/yetus-general-check/output/branch-spotbugs-hbase-common.txt |
| spotbugs | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/6/artifact/yetus-general-check/output/branch-spotbugs-hbase-client.txt |
| spotbugs | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/6/artifact/yetus-general-check/output/branch-spotbugs-hbase-zookeeper.txt |
| spotbugs | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/6/artifact/yetus-general-check/output/branch-spotbugs-hbase-http.txt |
| spotbugs | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/6/artifact/yetus-general-check/output/branch-spotbugs-hbase-procedure.txt |
| spotbugs | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/6/artifact/yetus-general-check/output/branch-spotbugs-hbase-server.txt |
| spotbugs | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/6/artifact/yetus-general-check/output/branch-spotbugs-hbase-mapreduce.txt |
| spotbugs | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/6/artifact/yetus-general-check/output/branch-spotbugs-hbase-thrift.txt |
| spotbugs | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/6/artifact/yetus-general-check/output/branch-spotbugs-hbase-rest.txt |
| spotbugs | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/6/artifact/yetus-general-check/output/branch-spotbugs-hbase-examples.txt |
| spotbugs | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/6/artifact/yetus-general-check/output/branch-spotbugs-hbase-hbtop.txt |
| spotbugs | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/6/artifact/yetus-general-check/output/branch-spotbugs-hbase-archetypes.txt |
| spotbugs | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/6/artifact/yetus-general-check/output/branch-spotbugs-root.txt |
| hadolint | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/6/artifact/yetus-general-check/output/diff-patch-hadolint.txt |
| Max. process+thread count | 93 (vs. ulimit of 10000) |
| modules | C: hbase-protocol-shaded hbase-common hbase-client hbase-zookeeper hbase-http hbase-procedure hbase-server hbase-mapreduce hbase-thrift hbase-shell hbase-rest hbase-examples hbase-shaded hbase-shaded/hbase-shaded-client hbase-shaded/hbase-shaded-client-byo-hadoop hbase-hbtop hbase-shaded/hbase-shaded-testing-util hbase-shaded/hbase-shaded-check-invariants hbase-shaded/hbase-shaded-with-hadoop-check-invariants hbase-archetypes . U: . |
| Console output | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/6/console |
| versions | git=2.17.1 maven=2018-06-17T18:33:14Z) shellcheck=0.4.6 hadolint=1.17.5-0-g443423c |
| Powered by | Apache Yetus 0.11.1 https://yetus.apache.org |
This message was automatically generated.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [hbase] Apache-HBase commented on issue #1265: HBASE-23077 move
entirely to spotbugs
Posted by GitBox <gi...@apache.org>.
Apache-HBase commented on issue #1265: HBASE-23077 move entirely to spotbugs
URL: https://github.com/apache/hbase/pull/1265#issuecomment-597428556
(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/3/console in case of problems.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [hbase] Apache-HBase commented on issue #1265: HBASE-23077 move
entirely to spotbugs
Posted by GitBox <gi...@apache.org>.
Apache-HBase commented on issue #1265: HBASE-23077 move entirely to spotbugs
URL: https://github.com/apache/hbase/pull/1265#issuecomment-597776689
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Comment |
|:----:|----------:|--------:|:--------|
| +0 :ok: | reexec | 1m 41s | Docker mode activated. |
| -0 :warning: | yetus | 0m 3s | Unprocessed flag(s): --brief-report-file --findbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck |
||| _ Prechecks _ |
||| _ master Compile Tests _ |
| +0 :ok: | mvndep | 0m 33s | Maven dependency ordering for branch |
| +1 :green_heart: | mvninstall | 6m 9s | master passed |
| +1 :green_heart: | compile | 3m 27s | master passed |
| +1 :green_heart: | shadedjars | 5m 16s | branch has no errors when building our shaded downstream artifacts. |
| +1 :green_heart: | javadoc | 8m 53s | master passed |
||| _ Patch Compile Tests _ |
| +0 :ok: | mvndep | 0m 19s | Maven dependency ordering for patch |
| +1 :green_heart: | mvninstall | 5m 50s | the patch passed |
| +1 :green_heart: | compile | 3m 24s | the patch passed |
| +1 :green_heart: | javac | 3m 24s | the patch passed |
| +1 :green_heart: | shadedjars | 5m 16s | patch has no errors when building our shaded downstream artifacts. |
| +1 :green_heart: | javadoc | 8m 44s | the patch passed |
||| _ Other Tests _ |
| -1 :x: | unit | 174m 16s | root in the patch failed. |
| | | 229m 30s | |
| Subsystem | Report/Notes |
|----------:|:-------------|
| Docker | Client=19.03.7 Server=19.03.7 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/6/artifact/yetus-jdk8-hadoop2-check/output/Dockerfile |
| GITHUB PR | https://github.com/apache/hbase/pull/1265 |
| Optional Tests | javac javadoc unit shadedjars compile |
| uname | Linux b8611fa6a39c 4.15.0-74-generic #84-Ubuntu SMP Thu Dec 19 08:06:28 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux |
| Build tool | maven |
| Personality | dev-support/hbase-personality.sh |
| git revision | master / d019fd2eed |
| Default Java | 1.8.0_232 |
| unit | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/6/artifact/yetus-jdk8-hadoop2-check/output/patch-unit-root.txt |
| Test Results | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/6/testReport/ |
| Max. process+thread count | 6069 (vs. ulimit of 10000) |
| modules | C: hbase-protocol-shaded hbase-common hbase-client hbase-zookeeper hbase-http hbase-procedure hbase-server hbase-mapreduce hbase-thrift hbase-shell hbase-rest hbase-examples hbase-shaded hbase-shaded/hbase-shaded-client hbase-shaded/hbase-shaded-client-byo-hadoop hbase-hbtop hbase-shaded/hbase-shaded-testing-util hbase-shaded/hbase-shaded-check-invariants hbase-shaded/hbase-shaded-with-hadoop-check-invariants hbase-archetypes . U: . |
| Console output | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/6/console |
| versions | git=2.17.1 maven=2018-06-17T18:33:14Z) |
| Powered by | Apache Yetus 0.11.1 https://yetus.apache.org |
This message was automatically generated.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [hbase] Apache-HBase commented on issue #1265: HBASE-23077 move
entirely to spotbugs
Posted by GitBox <gi...@apache.org>.
Apache-HBase commented on issue #1265: HBASE-23077 move entirely to spotbugs
URL: https://github.com/apache/hbase/pull/1265#issuecomment-597393988
(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1265/2/console in case of problems.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [hbase] busbey commented on a change in pull request #1265:
HBASE-23077 move entirely to spotbugs
Posted by GitBox <gi...@apache.org>.
busbey commented on a change in pull request #1265: HBASE-23077 move entirely to spotbugs
URL: https://github.com/apache/hbase/pull/1265#discussion_r390453465
##########
File path: hbase-shaded/hbase-shaded-client-byo-hadoop/pom.xml
##########
@@ -73,9 +73,9 @@
</activation>
<dependencies>
<dependency>
- <groupId>com.github.stephenc.findbugs</groupId>
- <artifactId>findbugs-annotations</artifactId>
- <optional>true</optional>
+ <groupId>com.github.stephenc.findbugs</groupId>
+ <artifactId>findbugs-annotations</artifactId>
+ <scope>provided</scope>
Review comment:
please label optional
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services