Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

HBASE-22883 Duplacate codes of method Threads.newDaemonThreadFactory(… #537

Merged
merged 1 commit into from
Aug 27, 2019
Merged

HBASE-22883 Duplacate codes of method Threads.newDaemonThreadFactory(… #537

merged 1 commit into from
Aug 27, 2019

Conversation

sunhelly
Copy link
Contributor

…) and class DaemonThreadFactory

@Apache-HBase
Copy link

💔 -1 overall

Vote Subsystem Runtime Comment
0 reexec 37 Docker mode activated.
_ Prechecks _
+1 dupname 0 No case conflicting files found.
+1 hbaseanti 0 Patch does not have any anti-patterns.
+1 @author 0 The patch does not contain any @author tags.
+1 test4tests 0 The patch appears to include 2 new or modified test files.
_ master Compile Tests _
0 mvndep 36 Maven dependency ordering for branch
+1 mvninstall 325 master passed
+1 compile 136 master passed
+1 checkstyle 138 master passed
+1 shadedjars 285 branch has no errors when building our shaded downstream artifacts.
+1 javadoc 92 master passed
0 spotbugs 250 Used deprecated FindBugs config; considering switching to SpotBugs.
+1 findbugs 347 master passed
_ Patch Compile Tests _
0 mvndep 17 Maven dependency ordering for patch
+1 mvninstall 329 the patch passed
+1 compile 149 the patch passed
+1 javac 149 the patch passed
-1 checkstyle 86 hbase-server: The patch generated 2 new + 61 unchanged - 2 fixed = 63 total (was 63)
+1 whitespace 0 The patch has no whitespace issues.
+1 shadedjars 276 patch has no errors when building our shaded downstream artifacts.
+1 hadoopcheck 1045 Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.
+1 javadoc 92 the patch passed
+1 findbugs 383 the patch passed
_ Other Tests _
+1 unit 192 hbase-common in the patch passed.
-1 unit 9340 hbase-server in the patch failed.
+1 unit 891 hbase-backup in the patch passed.
+1 unit 70 hbase-it in the patch passed.
+1 asflicense 127 The patch does not generate ASF License warnings.
14947
Reason Tests
Failed junit tests hadoop.hbase.master.assignment.TestOpenRegionProcedureHang
Subsystem Report/Notes
Docker Client=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-537/1/artifact/out/Dockerfile
GITHUB PR #537
Optional Tests dupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
uname Linux f24cfd50e56c 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 GNU/Linux
Build tool maven
Personality /home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-537/out/precommit/personality/provided.sh
git revision master / 1ea5d8b
Default Java 1.8.0_181
checkstyle https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-537/1/artifact/out/diff-checkstyle-hbase-server.txt
unit https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-537/1/artifact/out/patch-unit-hbase-server.txt
Test Results https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-537/1/testReport/
Max. process+thread count 4541 (vs. ulimit of 10000)
modules C: hbase-common hbase-server hbase-backup hbase-it U: .
Console output https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-537/1/console
versions git=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11
Powered by Apache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

@Apache-HBase
Copy link

💔 -1 overall

Vote Subsystem Runtime Comment
0 reexec 65 Docker mode activated.
_ Prechecks _
+1 dupname 0 No case conflicting files found.
+1 hbaseanti 0 Patch does not have any anti-patterns.
+1 @author 0 The patch does not contain any @author tags.
+1 test4tests 0 The patch appears to include 2 new or modified test files.
_ master Compile Tests _
0 mvndep 61 Maven dependency ordering for branch
+1 mvninstall 369 master passed
+1 compile 133 master passed
+1 checkstyle 147 master passed
+1 shadedjars 293 branch has no errors when building our shaded downstream artifacts.
+1 javadoc 83 master passed
0 spotbugs 254 Used deprecated FindBugs config; considering switching to SpotBugs.
+1 findbugs 345 master passed
_ Patch Compile Tests _
0 mvndep 15 Maven dependency ordering for patch
+1 mvninstall 325 the patch passed
+1 compile 132 the patch passed
+1 javac 132 the patch passed
-1 checkstyle 90 hbase-server: The patch generated 2 new + 61 unchanged - 2 fixed = 63 total (was 63)
+1 whitespace 0 The patch has no whitespace issues.
+1 shadedjars 294 patch has no errors when building our shaded downstream artifacts.
+1 hadoopcheck 1036 Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.
+1 javadoc 82 the patch passed
+1 findbugs 381 the patch passed
_ Other Tests _
+1 unit 173 hbase-common in the patch passed.
+1 unit 14561 hbase-server in the patch passed.
+1 unit 1362 hbase-backup in the patch passed.
+1 unit 89 hbase-it in the patch passed.
+1 asflicense 136 The patch does not generate ASF License warnings.
20672
Subsystem Report/Notes
Docker Client=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-537/2/artifact/out/Dockerfile
GITHUB PR #537
Optional Tests dupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
uname Linux 73d00232ad66 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 GNU/Linux
Build tool maven
Personality /home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-537/out/precommit/personality/provided.sh
git revision master / 0438fdc
Default Java 1.8.0_181
checkstyle https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-537/2/artifact/out/diff-checkstyle-hbase-server.txt
Test Results https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-537/2/testReport/
Max. process+thread count 5147 (vs. ulimit of 10000)
modules C: hbase-common hbase-server hbase-backup hbase-it U: .
Console output https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-537/2/console
versions git=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11
Powered by Apache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

@Apache-HBase
Copy link

🎊 +1 overall

Vote Subsystem Runtime Comment
0 reexec 38 Docker mode activated.
_ Prechecks _
+1 dupname 0 No case conflicting files found.
+1 hbaseanti 0 Patch does not have any anti-patterns.
+1 @author 0 The patch does not contain any @author tags.
+1 test4tests 0 The patch appears to include 2 new or modified test files.
_ master Compile Tests _
0 mvndep 37 Maven dependency ordering for branch
+1 mvninstall 310 master passed
+1 compile 134 master passed
+1 checkstyle 136 master passed
+1 shadedjars 268 branch has no errors when building our shaded downstream artifacts.
+1 javadoc 88 master passed
0 spotbugs 238 Used deprecated FindBugs config; considering switching to SpotBugs.
+1 findbugs 333 master passed
_ Patch Compile Tests _
0 mvndep 17 Maven dependency ordering for patch
+1 mvninstall 301 the patch passed
+1 compile 136 the patch passed
+1 javac 136 the patch passed
+1 checkstyle 26 The patch passed checkstyle in hbase-common
+1 checkstyle 80 hbase-server: The patch generated 0 new + 60 unchanged - 3 fixed = 60 total (was 63)
+1 checkstyle 15 The patch passed checkstyle in hbase-backup
+1 checkstyle 16 The patch passed checkstyle in hbase-it
+1 whitespace 0 The patch has no whitespace issues.
+1 shadedjars 272 patch has no errors when building our shaded downstream artifacts.
+1 hadoopcheck 927 Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.
+1 javadoc 88 the patch passed
+1 findbugs 344 the patch passed
_ Other Tests _
+1 unit 183 hbase-common in the patch passed.
+1 unit 8982 hbase-server in the patch passed.
+1 unit 876 hbase-backup in the patch passed.
+1 unit 67 hbase-it in the patch passed.
+1 asflicense 122 The patch does not generate ASF License warnings.
14217
Subsystem Report/Notes
Docker Client=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-537/3/artifact/out/Dockerfile
GITHUB PR #537
Optional Tests dupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
uname Linux 4e732f5caca1 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 GNU/Linux
Build tool maven
Personality /home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-537/out/precommit/personality/provided.sh
git revision master / c882672
Default Java 1.8.0_181
Test Results https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-537/3/testReport/
Max. process+thread count 5189 (vs. ulimit of 10000)
modules C: hbase-common hbase-server hbase-backup hbase-it U: .
Console output https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-537/3/console
versions git=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11
Powered by Apache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

@saintstack saintstack merged commit 3e2cfc1 into apache:master Aug 27, 2019
@Apache-HBase
Copy link

💔 -1 overall

Vote Subsystem Runtime Comment
0 reexec 154 Docker mode activated.
_ Prechecks _
+1 dupname 0 No case conflicting files found.
+1 hbaseanti 0 Patch does not have any anti-patterns.
+1 @author 0 The patch does not contain any @author tags.
+1 test4tests 0 The patch appears to include 2 new or modified test files.
_ master Compile Tests _
0 mvndep 44 Maven dependency ordering for branch
+1 mvninstall 475 master passed
+1 compile 164 master passed
+1 checkstyle 168 master passed
+1 shadedjars 364 branch has no errors when building our shaded downstream artifacts.
+1 javadoc 113 master passed
0 spotbugs 313 Used deprecated FindBugs config; considering switching to SpotBugs.
+1 findbugs 425 master passed
_ Patch Compile Tests _
0 mvndep 18 Maven dependency ordering for patch
+1 mvninstall 392 the patch passed
+1 compile 142 the patch passed
+1 javac 142 the patch passed
+1 checkstyle 26 The patch passed checkstyle in hbase-common
+1 checkstyle 95 hbase-server: The patch generated 0 new + 60 unchanged - 3 fixed = 60 total (was 63)
+1 checkstyle 13 The patch passed checkstyle in hbase-backup
+1 checkstyle 16 The patch passed checkstyle in hbase-it
+1 whitespace 0 The patch has no whitespace issues.
+1 shadedjars 315 patch has no errors when building our shaded downstream artifacts.
+1 hadoopcheck 1069 Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.
+1 javadoc 89 the patch passed
+1 findbugs 414 the patch passed
_ Other Tests _
+1 unit 182 hbase-common in the patch passed.
-1 unit 15851 hbase-server in the patch failed.
+1 unit 1365 hbase-backup in the patch passed.
+1 unit 93 hbase-it in the patch passed.
+1 asflicense 144 The patch does not generate ASF License warnings.
22623
Reason Tests
Failed junit tests hadoop.hbase.client.TestAsyncRegionAdminApi
hadoop.hbase.quotas.TestSpaceQuotaRemoval
Subsystem Report/Notes
Docker Client=18.09.7 Server=18.09.7 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-537/4/artifact/out/Dockerfile
GITHUB PR #537
Optional Tests dupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
uname Linux 8fce596f4bee 4.15.0-52-generic #56-Ubuntu SMP Tue Jun 4 22:49:08 UTC 2019 x86_64 GNU/Linux
Build tool maven
Personality /home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-537/out/precommit/personality/provided.sh
git revision master / 5106f28
Default Java 1.8.0_181
unit https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-537/4/artifact/out/patch-unit-hbase-server.txt
Test Results https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-537/4/testReport/
Max. process+thread count 4703 (vs. ulimit of 10000)
modules C: hbase-common hbase-server hbase-backup hbase-it U: .
Console output https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-537/4/console
versions git=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11
Powered by Apache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants