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/05/04 19:46:42 UTC
[GitHub] [hbase] Apache-HBase commented on pull request #1201: HBASE-23639 : Moving classes out of hbase-it /test for direct API use of chaos.
Apache-HBase commented on pull request #1201:
URL: https://github.com/apache/hbase/pull/1201#issuecomment-623668110
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Comment |
|:----:|----------:|--------:|:--------|
| +0 :ok: | reexec | 0m 27s | Docker mode activated. |
||| _ Prechecks _ |
| +1 :green_heart: | dupname | 0m 1s | No case conflicting files found. |
| +1 :green_heart: | hbaseanti | 0m 0s | Patch does not have any anti-patterns. |
| +1 :green_heart: | @author | 0m 0s | The patch does not contain any @author tags. |
||| _ master Compile Tests _ |
| +0 :ok: | mvndep | 0m 31s | Maven dependency ordering for branch |
| +1 :green_heart: | mvninstall | 3m 52s | master passed |
| +1 :green_heart: | checkstyle | 2m 22s | master passed |
| +1 :green_heart: | spotbugs | 13m 42s | master passed |
| -0 :warning: | patch | 2m 53s | Used diff version of patch file. Binary files and potentially other changes not applied. Please rebase and squash commits if necessary. |
||| _ Patch Compile Tests _ |
| +0 :ok: | mvndep | 0m 12s | Maven dependency ordering for patch |
| +1 :green_heart: | mvninstall | 3m 41s | the patch passed |
| -0 :warning: | checkstyle | 2m 22s | root: The patch generated 5 new + 314 unchanged - 31 fixed = 319 total (was 345) |
| -0 :warning: | whitespace | 0m 0s | The patch has 9 line(s) that end in whitespace. Use git apply --whitespace=fix <<patch_file>>. Refer https://git-scm.com/docs/git-apply |
| +1 :green_heart: | xml | 0m 5s | The patch has no ill-formed XML file. |
| +1 :green_heart: | hadoopcheck | 12m 0s | Patch does not cause any errors with Hadoop 3.1.2 3.2.1. |
| -1 :x: | spotbugs | 2m 21s | hbase-server generated 3 new + 0 unchanged - 0 fixed = 3 total (was 0) |
| -1 :x: | spotbugs | 9m 44s | root generated 4 new + 0 unchanged - 0 fixed = 4 total (was 0) |
||| _ Other Tests _ |
| +1 :green_heart: | asflicense | 1m 13s | The patch does not generate ASF License warnings. |
| | | 65m 22s | |
| Reason | Tests |
|-------:|:------|
| FindBugs | module:hbase-server |
| | org.apache.hadoop.hbase.HBaseTestingUtility.explainTableAvailability(TableName) concatenates strings using + in a loop At HBaseTestingUtility.java:in a loop At HBaseTestingUtility.java:[line 3255] |
| | Write to static field org.apache.hadoop.hbase.HBaseTestingUtility.FS_URI from instance method org.apache.hadoop.hbase.HBaseTestingUtility.setFileSystemURI(String) At HBaseTestingUtility.java:from instance method org.apache.hadoop.hbase.HBaseTestingUtility.setFileSystemURI(String) At HBaseTestingUtility.java:[line 4152] |
| | Write to static field org.apache.hadoop.hbase.MiniHBaseCluster.index from instance method org.apache.hadoop.hbase.MiniHBaseCluster.startMaster() At MiniHBaseCluster.java:from instance method org.apache.hadoop.hbase.MiniHBaseCluster.startMaster() At MiniHBaseCluster.java:[line 546] |
| FindBugs | module:root |
| | org.apache.hadoop.hbase.HBaseTestingUtility.explainTableAvailability(TableName) concatenates strings using + in a loop At HBaseTestingUtility.java:in a loop At HBaseTestingUtility.java:[line 3255] |
| | Write to static field org.apache.hadoop.hbase.HBaseTestingUtility.FS_URI from instance method org.apache.hadoop.hbase.HBaseTestingUtility.setFileSystemURI(String) At HBaseTestingUtility.java:from instance method org.apache.hadoop.hbase.HBaseTestingUtility.setFileSystemURI(String) At HBaseTestingUtility.java:[line 4152] |
| | Write to static field org.apache.hadoop.hbase.MiniHBaseCluster.index from instance method org.apache.hadoop.hbase.MiniHBaseCluster.startMaster() At MiniHBaseCluster.java:from instance method org.apache.hadoop.hbase.MiniHBaseCluster.startMaster() At MiniHBaseCluster.java:[line 546] |
| | org.apache.hadoop.hbase.chaos.factories.MonkeyConstants.MONKEY_CONFIGURATION_KEY_PREFIXES is a mutable collection At MonkeyConstants.java: At MonkeyConstants.java:[line 75] |
| Subsystem | Report/Notes |
|----------:|:-------------|
| Docker | Client=19.03.8 Server=19.03.8 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1201/7/artifact/yetus-general-check/output/Dockerfile |
| GITHUB PR | https://github.com/apache/hbase/pull/1201 |
| Optional Tests | dupname asflicense hadoopcheck xml spotbugs hbaseanti checkstyle |
| uname | Linux a3eca9e0ffd8 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 / e37aafcfc2 |
| checkstyle | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1201/7/artifact/yetus-general-check/output/diff-checkstyle-root.txt |
| whitespace | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1201/7/artifact/yetus-general-check/output/whitespace-eol.txt |
| spotbugs | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1201/7/artifact/yetus-general-check/output/new-spotbugs-hbase-server.html |
| spotbugs | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1201/7/artifact/yetus-general-check/output/new-spotbugs-root.html |
| Max. process+thread count | 123 (vs. ulimit of 12500) |
| modules | C: hbase-common hbase-hadoop-compat hbase-zookeeper hbase-http hbase-server hbase-it . U: . |
| Console output | https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1201/7/console |
| versions | git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f) spotbugs=3.1.12 |
| 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