Skip to content

HBASE-22231 Removed unused and '*' imports - #143

Merged
HorizonNet merged 1 commit into
apache:masterfrom
ultratendency:HBASE-22231
Apr 20, 2019
Merged

HBASE-22231 Removed unused and '*' imports#143
HorizonNet merged 1 commit into
apache:masterfrom
ultratendency:HBASE-22231

Conversation

@HorizonNet

Copy link
Copy Markdown
Contributor

No description provided.

@Apache-HBase

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
0reexec22Docker mode activated.
_ Prechecks _
+1hbaseanti0Patch does not have any anti-patterns.
+1@author0The patch does not contain any @author tags.
+1test4tests0The patch appears to include 58 new or modified test files.
_ master Compile Tests _
0mvndep27Maven dependency ordering for branch
+1mvninstall244master passed
+1compile194master passed
+1checkstyle201master passed
+1shadedjars267branch has no errors when building our shaded downstream artifacts.
+1findbugs435master passed
+1javadoc135master passed
_ Patch Compile Tests _
0mvndep15Maven dependency ordering for patch
+1mvninstall250the patch passed
+1compile214the patch passed
+1javac214the patch passed
+1checkstyle22hbase-common: The patch generated 0 new + 11 unchanged - 3 fixed = 11 total (was 14)
-1checkstyle30hbase-client: The patch generated 2 new + 22 unchanged - 16 fixed = 24 total (was 38)
+1checkstyle13hbase-procedure: The patch generated 0 new + 1 unchanged - 7 fixed = 1 total (was 8)
-1checkstyle75hbase-server: The patch generated 4 new + 258 unchanged - 79 fixed = 262 total (was 337)
+1checkstyle17hbase-mapreduce: The patch generated 0 new + 91 unchanged - 11 fixed = 91 total (was 102)
-1checkstyle15hbase-it: The patch generated 1 new + 90 unchanged - 7 fixed = 91 total (was 97)
+1checkstyle14hbase-rest: The patch generated 0 new + 15 unchanged - 2 fixed = 15 total (was 17)
+1whitespace0The patch has no whitespace issues.
+1shadedjars274patch has no errors when building our shaded downstream artifacts.
+1hadoopcheck498Patch does not cause any errors with Hadoop 2.7.4 or 3.0.0.
+1findbugs462the patch passed
+1javadoc143the patch passed
_ Other Tests _
+1unit159hbase-common in the patch passed.
+1unit192hbase-client in the patch passed.
+1unit216hbase-procedure in the patch passed.
+1unit7414hbase-server in the patch passed.
+1unit894hbase-mapreduce in the patch passed.
+1unit69hbase-it in the patch passed.
+1unit268hbase-rest in the patch passed.
+1asflicense239The patch does not generate ASF License warnings.
13189
SubsystemReport/Notes
DockerClient=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-143/2/artifact/out/Dockerfile
GITHUB PR#143
Optional Testsdupname asflicense javac javadoc unit findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
unameLinux 2e2da22f3a0c 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 17:16:02 UTC 2018 x86_64 GNU/Linux
Build toolmaven
Personality/testptch/patchprocess/precommit/personality/provided.sh
git revisionmaster / 268bcce
mavenversion: Apache Maven 3.5.4 (1edded0938998edf8bf061f1ceb3cfdeccf443fe; 2018-06-17T18:33:14Z)
Default Java1.8.0_181
findbugsv3.1.11
checkstylehttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-143/2/artifact/out/diff-checkstyle-hbase-client.txt
checkstylehttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-143/2/artifact/out/diff-checkstyle-hbase-server.txt
checkstylehttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-143/2/artifact/out/diff-checkstyle-hbase-it.txt
Test Resultshttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-143/2/testReport/
Max. process+thread count5379 (vs. ulimit of 10000)
modulesC: hbase-common hbase-client hbase-procedure hbase-server hbase-mapreduce hbase-it hbase-rest U: .
Console outputhttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-143/2/console
Powered byApache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
0reexec285Docker mode activated.
_ Prechecks _
+1hbaseanti0Patch does not have any anti-patterns.
+1@author0The patch does not contain any @author tags.
+1test4tests0The patch appears to include 56 new or modified test files.
_ master Compile Tests _
0mvndep30Maven dependency ordering for branch
+1mvninstall308master passed
+1compile220master passed
+1checkstyle221master passed
+1shadedjars331branch has no errors when building our shaded downstream artifacts.
+1findbugs517master passed
+1javadoc150master passed
_ Patch Compile Tests _
0mvndep16Maven dependency ordering for patch
+1mvninstall309the patch passed
+1compile221the patch passed
+1javac221the patch passed
+1checkstyle29hbase-common: The patch generated 0 new + 11 unchanged - 3 fixed = 11 total (was 14)
-1checkstyle39hbase-client: The patch generated 2 new + 22 unchanged - 16 fixed = 24 total (was 38)
-1checkstyle93hbase-server: The patch generated 4 new + 258 unchanged - 79 fixed = 262 total (was 337)
+1checkstyle23hbase-mapreduce: The patch generated 0 new + 91 unchanged - 11 fixed = 91 total (was 102)
-1checkstyle20hbase-it: The patch generated 1 new + 90 unchanged - 7 fixed = 91 total (was 97)
+1checkstyle19hbase-rest: The patch generated 0 new + 15 unchanged - 2 fixed = 15 total (was 17)
+1whitespace0The patch has no whitespace issues.
+1shadedjars336patch has no errors when building our shaded downstream artifacts.
+1hadoopcheck638Patch does not cause any errors with Hadoop 2.7.4 or 3.0.0.
+1findbugs577the patch passed
+1javadoc152the patch passed
_ Other Tests _
+1unit188hbase-common in the patch passed.
+1unit219hbase-client in the patch passed.
+1unit13821hbase-server in the patch passed.
+1unit1330hbase-mapreduce in the patch passed.
+1unit92hbase-it in the patch passed.
+1unit523hbase-rest in the patch passed.
+1asflicense198The patch does not generate ASF License warnings.
21088
SubsystemReport/Notes
DockerClient=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-143/3/artifact/out/Dockerfile
GITHUB PR#143
Optional Testsdupname asflicense javac javadoc unit findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
unameLinux 3cea82c2ae3d 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 17:16:02 UTC 2018 x86_64 GNU/Linux
Build toolmaven
Personality/testptch/patchprocess/precommit/personality/provided.sh
git revisionmaster / 5b01e61
mavenversion: Apache Maven 3.5.4 (1edded0938998edf8bf061f1ceb3cfdeccf443fe; 2018-06-17T18:33:14Z)
Default Java1.8.0_181
findbugsv3.1.11
checkstylehttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-143/3/artifact/out/diff-checkstyle-hbase-client.txt
checkstylehttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-143/3/artifact/out/diff-checkstyle-hbase-server.txt
checkstylehttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-143/3/artifact/out/diff-checkstyle-hbase-it.txt
Test Resultshttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-143/3/testReport/
Max. process+thread count5364 (vs. ulimit of 10000)
modulesC: hbase-common hbase-client hbase-server hbase-mapreduce hbase-it hbase-rest U: .
Console outputhttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-143/3/console
Powered byApache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
0reexec23Docker mode activated.
_ Prechecks _
+1hbaseanti0Patch does not have any anti-patterns.
+1@author0The patch does not contain any @author tags.
+1test4tests0The patch appears to include 56 new or modified test files.
_ master Compile Tests _
0mvndep28Maven dependency ordering for branch
+1mvninstall249master passed
+1compile176master passed
+1checkstyle176master passed
+1shadedjars253branch has no errors when building our shaded downstream artifacts.
+1findbugs375master passed
+1javadoc126master passed
_ Patch Compile Tests _
0mvndep16Maven dependency ordering for patch
+1mvninstall254the patch passed
+1compile191the patch passed
+1javac191the patch passed
+1checkstyle20hbase-common: The patch generated 0 new + 11 unchanged - 3 fixed = 11 total (was 14)
+1checkstyle30hbase-client: The patch generated 0 new + 21 unchanged - 17 fixed = 21 total (was 38)
+1checkstyle67hbase-server: The patch generated 0 new + 255 unchanged - 82 fixed = 255 total (was 337)
+1checkstyle17hbase-mapreduce: The patch generated 0 new + 91 unchanged - 11 fixed = 91 total (was 102)
+1checkstyle17hbase-it: The patch generated 0 new + 88 unchanged - 9 fixed = 88 total (was 97)
+1checkstyle15hbase-rest: The patch generated 0 new + 15 unchanged - 2 fixed = 15 total (was 17)
+1whitespace0The patch has no whitespace issues.
+1shadedjars261patch has no errors when building our shaded downstream artifacts.
+1hadoopcheck499Patch does not cause any errors with Hadoop 2.7.4 or 3.0.0.
+1findbugs433the patch passed
+1javadoc126the patch passed
_ Other Tests _
+1unit160hbase-common in the patch passed.
+1unit194hbase-client in the patch passed.
-1unit7640hbase-server in the patch failed.
-1unit920hbase-mapreduce in the patch failed.
+1unit60hbase-it in the patch passed.
+1unit268hbase-rest in the patch passed.
+1asflicense145The patch does not generate ASF License warnings.
12893
ReasonTests
Failed junit testshadoop.hbase.master.procedure.TestSCP
SubsystemReport/Notes
DockerClient=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-143/4/artifact/out/Dockerfile
GITHUB PR#143
Optional Testsdupname asflicense javac javadoc unit findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
unameLinux 2baf919fe711 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 17:16:02 UTC 2018 x86_64 GNU/Linux
Build toolmaven
Personality/testptch/patchprocess/precommit/personality/provided.sh
git revisionmaster / 5b01e61
mavenversion: Apache Maven 3.5.4 (1edded0938998edf8bf061f1ceb3cfdeccf443fe; 2018-06-17T18:33:14Z)
Default Java1.8.0_181
findbugsv3.1.11
unithttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-143/4/artifact/out/patch-unit-hbase-server.txt
unithttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-143/4/artifact/out/patch-unit-hbase-mapreduce.txt
Test Resultshttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-143/4/testReport/
Max. process+thread count5292 (vs. ulimit of 10000)
modulesC: hbase-common hbase-client hbase-server hbase-mapreduce hbase-it hbase-rest U: .
Console outputhttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-143/4/console
Powered byApache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

@HorizonNet
HorizonNet merged commit f30d6c9 into apache:masterApr 20, 2019
@HorizonNet
HorizonNet deleted the HBASE-22231 branch April 20, 2019 20:03
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@HorizonNet@Apache-HBase@petersomogyi