Skip to content

Conversation

@liuml07
Copy link
Member

@liuml07 liuml07 commented Jul 18, 2020

@Apache-HBase
Copy link

🎊 +1 overall

Vote Subsystem Runtime Comment
+0 🆗 reexec 2m 13s Docker mode activated.
_ Prechecks _
+1 💚 dupname 0m 0s No case conflicting files found.
+1 💚 hbaseanti 0m 0s Patch does not have any anti-patterns.
+1 💚 @author 0m 0s The patch does not contain any @author tags.
_ master Compile Tests _
+0 🆗 mvndep 0m 31s Maven dependency ordering for branch
+1 💚 mvninstall 4m 52s master passed
+1 💚 checkstyle 2m 25s master passed
+1 💚 spotbugs 5m 5s master passed
_ Patch Compile Tests _
+0 🆗 mvndep 0m 15s Maven dependency ordering for patch
+1 💚 mvninstall 4m 34s the patch passed
+1 💚 checkstyle 2m 38s the patch passed
+1 💚 whitespace 0m 0s The patch has no whitespace issues.
+1 💚 hadoopcheck 15m 16s Patch does not cause any errors with Hadoop 3.1.2 3.2.1.
+1 💚 spotbugs 5m 48s the patch passed
_ Other Tests _
+1 💚 asflicense 0m 40s The patch does not generate ASF License warnings.
54m 35s
Subsystem Report/Notes
Docker Client=19.03.12 Server=19.03.12 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-2087/1/artifact/yetus-general-check/output/Dockerfile
GITHUB PR #2087
Optional Tests dupname asflicense spotbugs hadoopcheck hbaseanti checkstyle
uname Linux b164c8422e17 4.15.0-101-generic #102-Ubuntu SMP Mon May 11 10:07:26 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality dev-support/hbase-personality.sh
git revision master / 9b02a26
Max. process+thread count 84 (vs. ulimit of 12500)
modules C: hbase-server hbase-thrift hbase-rest U: .
Console output https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-2087/1/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.

Copy link
Contributor

@virajjasani virajjasani left a comment

Choose a reason for hiding this comment

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

small nit, looks good otherwise.

* Return the current JVM version information.
*/
public static String getVersion() {
return System.getProperty("java.vm.vendor", "UsNKNOWN_VM_VENDOR") + ' ' +
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: typo UNKNOWN_VM_VENDOR ?

@Apache-HBase
Copy link

💔 -1 overall

Vote Subsystem Runtime Comment
+0 🆗 reexec 0m 35s Docker mode activated.
-0 ⚠️ 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 🆗 mvndep 0m 23s Maven dependency ordering for branch
+1 💚 mvninstall 4m 16s master passed
+1 💚 compile 2m 15s master passed
+1 💚 shadedjars 5m 56s branch has no errors when building our shaded downstream artifacts.
-0 ⚠️ javadoc 0m 22s hbase-rest in master failed.
-0 ⚠️ javadoc 0m 40s hbase-server in master failed.
-0 ⚠️ javadoc 0m 52s hbase-thrift in master failed.
_ Patch Compile Tests _
+0 🆗 mvndep 0m 14s Maven dependency ordering for patch
+1 💚 mvninstall 4m 8s the patch passed
+1 💚 compile 2m 11s the patch passed
+1 💚 javac 2m 11s the patch passed
+1 💚 shadedjars 6m 14s patch has no errors when building our shaded downstream artifacts.
-0 ⚠️ javadoc 0m 52s hbase-server in the patch failed.
-0 ⚠️ javadoc 1m 8s hbase-thrift in the patch failed.
-0 ⚠️ javadoc 0m 24s hbase-rest in the patch failed.
_ Other Tests _
-1 ❌ unit 148m 8s hbase-server in the patch failed.
+1 💚 unit 5m 5s hbase-thrift in the patch passed.
+1 💚 unit 3m 27s hbase-rest in the patch passed.
189m 27s
Subsystem Report/Notes
Docker Client=19.03.12 Server=19.03.12 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-2087/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile
GITHUB PR #2087
Optional Tests javac javadoc unit shadedjars compile
uname Linux dba1d203d7b0 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 / 9b02a26
Default Java 2020-01-14
javadoc https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-2087/1/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-rest.txt
javadoc https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-2087/1/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-server.txt
javadoc https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-2087/1/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-thrift.txt
javadoc https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-2087/1/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-server.txt
javadoc https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-2087/1/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-thrift.txt
javadoc https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-2087/1/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-rest.txt
unit https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-2087/1/artifact/yetus-jdk11-hadoop3-check/output/patch-unit-hbase-server.txt
Test Results https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-2087/1/testReport/
Max. process+thread count 4094 (vs. ulimit of 12500)
modules C: hbase-server hbase-thrift hbase-rest U: .
Console output https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-2087/1/console
versions git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f)
Powered by Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase
Copy link

💔 -1 overall

Vote Subsystem Runtime Comment
+0 🆗 reexec 0m 31s Docker mode activated.
-0 ⚠️ 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 🆗 mvndep 0m 24s Maven dependency ordering for branch
+1 💚 mvninstall 3m 43s master passed
+1 💚 compile 1m 55s master passed
+1 💚 shadedjars 5m 37s branch has no errors when building our shaded downstream artifacts.
+1 💚 javadoc 1m 29s master passed
_ Patch Compile Tests _
+0 🆗 mvndep 0m 14s Maven dependency ordering for patch
+1 💚 mvninstall 3m 32s the patch passed
+1 💚 compile 1m 51s the patch passed
+1 💚 javac 1m 51s the patch passed
+1 💚 shadedjars 5m 35s patch has no errors when building our shaded downstream artifacts.
+1 💚 javadoc 1m 26s the patch passed
_ Other Tests _
-1 ❌ unit 160m 14s hbase-server in the patch failed.
+1 💚 unit 4m 59s hbase-thrift in the patch passed.
+1 💚 unit 3m 22s hbase-rest in the patch passed.
197m 12s
Subsystem Report/Notes
Docker Client=19.03.12 Server=19.03.12 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-2087/1/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile
GITHUB PR #2087
Optional Tests javac javadoc unit shadedjars compile
uname Linux 9d7a0ba5c276 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 / 9b02a26
Default Java 1.8.0_232
unit https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-2087/1/artifact/yetus-jdk8-hadoop3-check/output/patch-unit-hbase-server.txt
Test Results https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-2087/1/testReport/
Max. process+thread count 4095 (vs. ulimit of 12500)
modules C: hbase-server hbase-thrift hbase-rest U: .
Console output https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-2087/1/console
versions git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f)
Powered by Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase
Copy link

🎊 +1 overall

Vote Subsystem Runtime Comment
+0 🆗 reexec 1m 41s Docker mode activated.
_ Prechecks _
+1 💚 dupname 0m 0s No case conflicting files found.
+1 💚 hbaseanti 0m 0s Patch does not have any anti-patterns.
+1 💚 @author 0m 0s The patch does not contain any @author tags.
_ master Compile Tests _
+0 🆗 mvndep 0m 19s Maven dependency ordering for branch
+1 💚 mvninstall 4m 7s master passed
+1 💚 checkstyle 2m 12s master passed
+1 💚 spotbugs 4m 23s master passed
_ Patch Compile Tests _
+0 🆗 mvndep 0m 13s Maven dependency ordering for patch
+1 💚 mvninstall 3m 56s the patch passed
+1 💚 checkstyle 2m 10s the patch passed
+1 💚 whitespace 0m 0s The patch has no whitespace issues.
+1 💚 hadoopcheck 12m 42s Patch does not cause any errors with Hadoop 3.1.2 3.2.1.
+1 💚 spotbugs 5m 54s the patch passed
_ Other Tests _
+1 💚 asflicense 0m 33s The patch does not generate ASF License warnings.
47m 16s
Subsystem Report/Notes
Docker Client=19.03.12 Server=19.03.12 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-2087/2/artifact/yetus-general-check/output/Dockerfile
GITHUB PR #2087
Optional Tests dupname asflicense spotbugs hadoopcheck hbaseanti checkstyle
uname Linux 5d092eac02bd 4.15.0-101-generic #102-Ubuntu SMP Mon May 11 10:07:26 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality dev-support/hbase-personality.sh
git revision master / 9b02a26
Max. process+thread count 85 (vs. ulimit of 12500)
modules C: hbase-server hbase-thrift hbase-rest U: .
Console output https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-2087/2/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.

@Apache-HBase
Copy link

🎊 +1 overall

Vote Subsystem Runtime Comment
+0 🆗 reexec 0m 30s Docker mode activated.
-0 ⚠️ 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 🆗 mvndep 0m 15s Maven dependency ordering for branch
+1 💚 mvninstall 3m 57s master passed
+1 💚 compile 2m 6s master passed
+1 💚 shadedjars 5m 44s branch has no errors when building our shaded downstream artifacts.
-0 ⚠️ javadoc 0m 40s hbase-server in master failed.
-0 ⚠️ javadoc 0m 49s hbase-thrift in master failed.
-0 ⚠️ javadoc 0m 21s hbase-rest in master failed.
_ Patch Compile Tests _
+0 🆗 mvndep 0m 15s Maven dependency ordering for patch
+1 💚 mvninstall 4m 0s the patch passed
+1 💚 compile 2m 9s the patch passed
+1 💚 javac 2m 9s the patch passed
+1 💚 shadedjars 5m 45s patch has no errors when building our shaded downstream artifacts.
-0 ⚠️ javadoc 0m 39s hbase-server in the patch failed.
-0 ⚠️ javadoc 0m 49s hbase-thrift in the patch failed.
-0 ⚠️ javadoc 0m 20s hbase-rest in the patch failed.
_ Other Tests _
+1 💚 unit 128m 50s hbase-server in the patch passed.
+1 💚 unit 4m 48s hbase-thrift in the patch passed.
+1 💚 unit 2m 51s hbase-rest in the patch passed.
167m 11s
Subsystem Report/Notes
Docker Client=19.03.12 Server=19.03.12 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-2087/2/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile
GITHUB PR #2087
Optional Tests javac javadoc unit shadedjars compile
uname Linux 2b0ae9f0451c 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 / 9b02a26
Default Java 2020-01-14
javadoc https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-2087/2/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-server.txt
javadoc https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-2087/2/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-thrift.txt
javadoc https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-2087/2/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-rest.txt
javadoc https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-2087/2/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-server.txt
javadoc https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-2087/2/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-thrift.txt
javadoc https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-2087/2/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-rest.txt
Test Results https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-2087/2/testReport/
Max. process+thread count 4537 (vs. ulimit of 12500)
modules C: hbase-server hbase-thrift hbase-rest U: .
Console output https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-2087/2/console
versions git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f)
Powered by Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase
Copy link

🎊 +1 overall

Vote Subsystem Runtime Comment
+0 🆗 reexec 1m 40s Docker mode activated.
-0 ⚠️ 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 🆗 mvndep 0m 24s Maven dependency ordering for branch
+1 💚 mvninstall 4m 7s master passed
+1 💚 compile 2m 0s master passed
+1 💚 shadedjars 6m 23s branch has no errors when building our shaded downstream artifacts.
+1 💚 javadoc 1m 33s master passed
_ Patch Compile Tests _
+0 🆗 mvndep 0m 14s Maven dependency ordering for patch
+1 💚 mvninstall 3m 56s the patch passed
+1 💚 compile 2m 2s the patch passed
+1 💚 javac 2m 2s the patch passed
+1 💚 shadedjars 6m 7s patch has no errors when building our shaded downstream artifacts.
+1 💚 javadoc 1m 29s the patch passed
_ Other Tests _
+1 💚 unit 204m 4s hbase-server in the patch passed.
+1 💚 unit 5m 34s hbase-thrift in the patch passed.
+1 💚 unit 3m 55s hbase-rest in the patch passed.
245m 23s
Subsystem Report/Notes
Docker Client=19.03.12 Server=19.03.12 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-2087/2/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile
GITHUB PR #2087
Optional Tests javac javadoc unit shadedjars compile
uname Linux 4aba753ef16a 4.15.0-101-generic #102-Ubuntu SMP Mon May 11 10:07:26 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality dev-support/hbase-personality.sh
git revision master / 9b02a26
Default Java 1.8.0_232
Test Results https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-2087/2/testReport/
Max. process+thread count 3551 (vs. ulimit of 12500)
modules C: hbase-server hbase-thrift hbase-rest U: .
Console output https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-2087/2/console
versions git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f)
Powered by Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

Copy link
Contributor

@virajjasani virajjasani left a comment

Choose a reason for hiding this comment

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

+1

/**
* Return the current JVM version information.
*/
public static String getVersion() {
Copy link
Member

Choose a reason for hiding this comment

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

According to the javadoc in the class header, this is a repurposing of the existing class. I think it's fine to put this method into this class, as the name and location of the class seem like a reasonable place to find this information. Can you update the javadoc on the class header according to the expanded scope of the class?

Otherwise, +1.

@Apache-HBase
Copy link

🎊 +1 overall

Vote Subsystem Runtime Comment
+0 🆗 reexec 1m 11s Docker mode activated.
_ Prechecks _
+1 💚 dupname 0m 0s No case conflicting files found.
+1 💚 hbaseanti 0m 0s Patch does not have any anti-patterns.
+1 💚 @author 0m 0s The patch does not contain any @author tags.
_ master Compile Tests _
+0 🆗 mvndep 0m 30s Maven dependency ordering for branch
+1 💚 mvninstall 3m 53s master passed
+1 💚 checkstyle 2m 7s master passed
+1 💚 spotbugs 4m 4s master passed
_ Patch Compile Tests _
+0 🆗 mvndep 0m 12s Maven dependency ordering for patch
+1 💚 mvninstall 3m 45s the patch passed
+1 💚 checkstyle 2m 8s the patch passed
+1 💚 whitespace 0m 0s The patch has no whitespace issues.
+1 💚 hadoopcheck 12m 9s Patch does not cause any errors with Hadoop 3.1.2 3.2.1.
+1 💚 spotbugs 4m 38s the patch passed
_ Other Tests _
+1 💚 asflicense 0m 31s The patch does not generate ASF License warnings.
43m 17s
Subsystem Report/Notes
Docker Client=19.03.12 Server=19.03.12 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-2087/3/artifact/yetus-general-check/output/Dockerfile
GITHUB PR #2087
Optional Tests dupname asflicense spotbugs hadoopcheck hbaseanti checkstyle
uname Linux 981fdf8221c2 4.15.0-101-generic #102-Ubuntu SMP Mon May 11 10:07:26 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality dev-support/hbase-personality.sh
git revision master / a857c6d
Max. process+thread count 84 (vs. ulimit of 12500)
modules C: hbase-server hbase-thrift hbase-rest U: .
Console output https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-2087/3/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.

@Apache-HBase
Copy link

🎊 +1 overall

Vote Subsystem Runtime Comment
+0 🆗 reexec 0m 30s Docker mode activated.
-0 ⚠️ 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 🆗 mvndep 0m 23s Maven dependency ordering for branch
+1 💚 mvninstall 4m 10s master passed
+1 💚 compile 2m 7s master passed
+1 💚 shadedjars 5m 47s branch has no errors when building our shaded downstream artifacts.
-0 ⚠️ javadoc 0m 23s hbase-rest in master failed.
-0 ⚠️ javadoc 0m 40s hbase-server in master failed.
-0 ⚠️ javadoc 0m 52s hbase-thrift in master failed.
_ Patch Compile Tests _
+0 🆗 mvndep 0m 17s Maven dependency ordering for patch
+1 💚 mvninstall 4m 2s the patch passed
+1 💚 compile 2m 7s the patch passed
+1 💚 javac 2m 7s the patch passed
+1 💚 shadedjars 5m 43s patch has no errors when building our shaded downstream artifacts.
-0 ⚠️ javadoc 0m 41s hbase-server in the patch failed.
-0 ⚠️ javadoc 0m 51s hbase-thrift in the patch failed.
-0 ⚠️ javadoc 0m 20s hbase-rest in the patch failed.
_ Other Tests _
+1 💚 unit 130m 31s hbase-server in the patch passed.
+1 💚 unit 4m 7s hbase-thrift in the patch passed.
+1 💚 unit 3m 14s hbase-rest in the patch passed.
169m 24s
Subsystem Report/Notes
Docker Client=19.03.12 Server=19.03.12 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-2087/3/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile
GITHUB PR #2087
Optional Tests javac javadoc unit shadedjars compile
uname Linux 9451d1ee7f3b 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 / a857c6d
Default Java 2020-01-14
javadoc https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-2087/3/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-rest.txt
javadoc https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-2087/3/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-server.txt
javadoc https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-2087/3/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-thrift.txt
javadoc https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-2087/3/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-server.txt
javadoc https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-2087/3/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-thrift.txt
javadoc https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-2087/3/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-rest.txt
Test Results https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-2087/3/testReport/
Max. process+thread count 4325 (vs. ulimit of 12500)
modules C: hbase-server hbase-thrift hbase-rest U: .
Console output https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-2087/3/console
versions git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f)
Powered by Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase
Copy link

💔 -1 overall

Vote Subsystem Runtime Comment
+0 🆗 reexec 0m 38s Docker mode activated.
-0 ⚠️ 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 🆗 mvndep 0m 31s Maven dependency ordering for branch
+1 💚 mvninstall 4m 30s master passed
+1 💚 compile 2m 17s master passed
+1 💚 shadedjars 7m 23s branch has no errors when building our shaded downstream artifacts.
+1 💚 javadoc 1m 45s master passed
_ Patch Compile Tests _
+0 🆗 mvndep 0m 17s Maven dependency ordering for patch
+1 💚 mvninstall 4m 23s the patch passed
+1 💚 compile 2m 7s the patch passed
+1 💚 javac 2m 7s the patch passed
+1 💚 shadedjars 6m 39s patch has no errors when building our shaded downstream artifacts.
+1 💚 javadoc 1m 40s the patch passed
_ Other Tests _
-1 ❌ unit 147m 56s hbase-server in the patch failed.
+1 💚 unit 5m 2s hbase-thrift in the patch passed.
+1 💚 unit 3m 23s hbase-rest in the patch passed.
191m 18s
Subsystem Report/Notes
Docker Client=19.03.12 Server=19.03.12 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-2087/3/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile
GITHUB PR #2087
Optional Tests javac javadoc unit shadedjars compile
uname Linux 6f3ed1733217 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 / a857c6d
Default Java 1.8.0_232
unit https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-2087/3/artifact/yetus-jdk8-hadoop3-check/output/patch-unit-hbase-server.txt
Test Results https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-2087/3/testReport/
Max. process+thread count 4712 (vs. ulimit of 12500)
modules C: hbase-server hbase-thrift hbase-rest U: .
Console output https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-2087/3/console
versions git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f)
Powered by Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase
Copy link

🎊 +1 overall

Vote Subsystem Runtime Comment
+0 🆗 reexec 2m 11s Docker mode activated.
_ Prechecks _
+1 💚 dupname 0m 0s No case conflicting files found.
+1 💚 hbaseanti 0m 0s Patch does not have any anti-patterns.
+1 💚 @author 0m 0s The patch does not contain any @author tags.
_ master Compile Tests _
+0 🆗 mvndep 0m 24s Maven dependency ordering for branch
+1 💚 mvninstall 4m 13s master passed
+1 💚 checkstyle 2m 10s master passed
+1 💚 spotbugs 4m 48s master passed
_ Patch Compile Tests _
+0 🆗 mvndep 0m 15s Maven dependency ordering for patch
+1 💚 mvninstall 4m 10s the patch passed
+1 💚 checkstyle 2m 15s the patch passed
+1 💚 whitespace 0m 0s The patch has no whitespace issues.
+1 💚 hadoopcheck 12m 45s Patch does not cause any errors with Hadoop 3.1.2 3.2.1.
+1 💚 spotbugs 5m 4s the patch passed
_ Other Tests _
+1 💚 asflicense 0m 34s The patch does not generate ASF License warnings.
47m 14s
Subsystem Report/Notes
Docker Client=19.03.12 Server=19.03.12 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-2087/4/artifact/yetus-general-check/output/Dockerfile
GITHUB PR #2087
Optional Tests dupname asflicense spotbugs hadoopcheck hbaseanti checkstyle
uname Linux f487303a18ba 4.15.0-101-generic #102-Ubuntu SMP Mon May 11 10:07:26 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality dev-support/hbase-personality.sh
git revision master / 430602a
Max. process+thread count 84 (vs. ulimit of 12500)
modules C: hbase-server hbase-thrift hbase-rest U: .
Console output https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-2087/4/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.

@Apache-HBase
Copy link

🎊 +1 overall

Vote Subsystem Runtime Comment
+0 🆗 reexec 0m 32s Docker mode activated.
-0 ⚠️ 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 🆗 mvndep 0m 30s Maven dependency ordering for branch
+1 💚 mvninstall 4m 13s master passed
+1 💚 compile 2m 15s master passed
+1 💚 shadedjars 5m 48s branch has no errors when building our shaded downstream artifacts.
-0 ⚠️ javadoc 0m 44s hbase-server in master failed.
-0 ⚠️ javadoc 0m 53s hbase-thrift in master failed.
-0 ⚠️ javadoc 0m 20s hbase-rest in master failed.
_ Patch Compile Tests _
+0 🆗 mvndep 0m 14s Maven dependency ordering for patch
+1 💚 mvninstall 4m 16s the patch passed
+1 💚 compile 2m 11s the patch passed
+1 💚 javac 2m 11s the patch passed
+1 💚 shadedjars 6m 19s patch has no errors when building our shaded downstream artifacts.
-0 ⚠️ javadoc 0m 51s hbase-server in the patch failed.
-0 ⚠️ javadoc 1m 11s hbase-thrift in the patch failed.
-0 ⚠️ javadoc 0m 23s hbase-rest in the patch failed.
_ Other Tests _
+1 💚 unit 148m 59s hbase-server in the patch passed.
+1 💚 unit 5m 5s hbase-thrift in the patch passed.
+1 💚 unit 3m 12s hbase-rest in the patch passed.
190m 1s
Subsystem Report/Notes
Docker Client=19.03.12 Server=19.03.12 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-2087/4/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile
GITHUB PR #2087
Optional Tests javac javadoc unit shadedjars compile
uname Linux db94ff80c926 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 / 430602a
Default Java 2020-01-14
javadoc https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-2087/4/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-server.txt
javadoc https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-2087/4/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-thrift.txt
javadoc https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-2087/4/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-rest.txt
javadoc https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-2087/4/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-server.txt
javadoc https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-2087/4/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-thrift.txt
javadoc https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-2087/4/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-rest.txt
Test Results https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-2087/4/testReport/
Max. process+thread count 3882 (vs. ulimit of 12500)
modules C: hbase-server hbase-thrift hbase-rest U: .
Console output https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-2087/4/console
versions git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f)
Powered by Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase
Copy link

🎊 +1 overall

Vote Subsystem Runtime Comment
+0 🆗 reexec 0m 32s Docker mode activated.
-0 ⚠️ yetus 0m 2s Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck
_ Prechecks _
_ master Compile Tests _
+0 🆗 mvndep 0m 20s Maven dependency ordering for branch
+1 💚 mvninstall 3m 29s master passed
+1 💚 compile 1m 52s master passed
+1 💚 shadedjars 5m 41s branch has no errors when building our shaded downstream artifacts.
+1 💚 javadoc 1m 24s master passed
_ Patch Compile Tests _
+0 🆗 mvndep 0m 15s Maven dependency ordering for patch
+1 💚 mvninstall 3m 36s the patch passed
+1 💚 compile 1m 51s the patch passed
+1 💚 javac 1m 51s the patch passed
+1 💚 shadedjars 5m 36s patch has no errors when building our shaded downstream artifacts.
+1 💚 javadoc 1m 26s the patch passed
_ Other Tests _
+1 💚 unit 159m 33s hbase-server in the patch passed.
+1 💚 unit 5m 7s hbase-thrift in the patch passed.
+1 💚 unit 3m 27s hbase-rest in the patch passed.
196m 26s
Subsystem Report/Notes
Docker Client=19.03.12 Server=19.03.12 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-2087/4/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile
GITHUB PR #2087
Optional Tests javac javadoc unit shadedjars compile
uname Linux ce1cb607bfd9 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 / 430602a
Default Java 1.8.0_232
Test Results https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-2087/4/testReport/
Max. process+thread count 4106 (vs. ulimit of 12500)
modules C: hbase-server hbase-thrift hbase-rest U: .
Console output https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-2087/4/console
versions git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f)
Powered by Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

virajjasani pushed a commit that referenced this pull request Jul 21, 2020
…tes"

Closes #2087

Signed-off-by: Viraj Jasani <[email protected]>
Signed-off-by: Nick Dimiduk <[email protected]>
virajjasani pushed a commit that referenced this pull request Jul 21, 2020
…tes"

Closes #2087

Signed-off-by: Viraj Jasani <[email protected]>
Signed-off-by: Nick Dimiduk <[email protected]>
clarax pushed a commit to clarax/hbase that referenced this pull request Nov 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants