Skip to content

Comments

HBASE-22317 Support reading from meta replicas#194

Merged
Apache9 merged 1 commit intoapache:masterfrom
Apache9:HBASE-22317
Apr 28, 2019
Merged

HBASE-22317 Support reading from meta replicas#194
Apache9 merged 1 commit intoapache:masterfrom
Apache9:HBASE-22317

Conversation

@Apache9
Copy link
Contributor

@Apache9 Apache9 commented Apr 27, 2019

Also done some cleanups on TestMetaWithReplicas. Planed to reuse the test but finally give up, but I think the cleanup is still needed. Introduced a TestAsyncTableUseMetaReplicas instead.

@Apache-HBase
Copy link

💔 -1 overall

Vote Subsystem Runtime Comment
0 reexec 262 Docker mode activated.
_ Prechecks _
+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 8 new or modified test files.
_ master Compile Tests _
0 mvndep 30 Maven dependency ordering for branch
+1 mvninstall 249 master passed
+1 compile 76 master passed
+1 checkstyle 98 master passed
+1 shadedjars 259 branch has no errors when building our shaded downstream artifacts.
+1 findbugs 273 master passed
+1 javadoc 55 master passed
_ Patch Compile Tests _
0 mvndep 15 Maven dependency ordering for patch
+1 mvninstall 234 the patch passed
+1 compile 74 the patch passed
+1 javac 74 the patch passed
-1 checkstyle 68 hbase-server: The patch generated 2 new + 15 unchanged - 0 fixed = 17 total (was 15)
+1 whitespace 0 The patch has no whitespace issues.
+1 shadedjars 253 patch has no errors when building our shaded downstream artifacts.
+1 hadoopcheck 491 Patch does not cause any errors with Hadoop 2.7.4 or 3.0.0.
+1 findbugs 293 the patch passed
+1 javadoc 56 the patch passed
_ Other Tests _
+1 unit 202 hbase-client in the patch passed.
-1 unit 14979 hbase-server in the patch failed.
+1 asflicense 55 The patch does not generate ASF License warnings.
18147
Reason Tests
Failed junit tests hadoop.hbase.client.TestFromClientSide3
hadoop.hbase.client.TestSnapshotTemporaryDirectoryWithRegionReplicas
hadoop.hbase.client.TestMobSnapshotCloneIndependence
Subsystem Report/Notes
Docker Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-194/1/artifact/out/Dockerfile
GITHUB PR #194
Optional Tests dupname asflicense javac javadoc unit findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
uname Linux 2acc8cc8bcdb 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 17:16:02 UTC 2018 x86_64 GNU/Linux
Build tool maven
Personality /testptch/patchprocess/precommit/personality/provided.sh
git revision master / 962585d
maven version: Apache Maven 3.5.4 (1edded0938998edf8bf061f1ceb3cfdeccf443fe; 2018-06-17T18:33:14Z)
Default Java 1.8.0_181
findbugs v3.1.11
checkstyle https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-194/1/artifact/out/diff-checkstyle-hbase-server.txt
unit https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-194/1/artifact/out/patch-unit-hbase-server.txt
Test Results https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-194/1/testReport/
Max. process+thread count 5008 (vs. ulimit of 10000)
modules C: hbase-client hbase-server U: .
Console output https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-194/1/console
Powered by Apache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

@Apache-HBase
Copy link

💔 -1 overall

Vote Subsystem Runtime Comment
0 reexec 48 Docker mode activated.
_ Prechecks _
+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 8 new or modified test files.
_ master Compile Tests _
0 mvndep 30 Maven dependency ordering for branch
+1 mvninstall 302 master passed
+1 compile 98 master passed
+1 checkstyle 120 master passed
+1 shadedjars 322 branch has no errors when building our shaded downstream artifacts.
+1 findbugs 375 master passed
+1 javadoc 68 master passed
_ Patch Compile Tests _
0 mvndep 17 Maven dependency ordering for patch
+1 mvninstall 293 the patch passed
+1 compile 99 the patch passed
+1 javac 99 the patch passed
+1 checkstyle 126 the patch passed
+1 whitespace 0 The patch has no whitespace issues.
+1 shadedjars 326 patch has no errors when building our shaded downstream artifacts.
+1 hadoopcheck 626 Patch does not cause any errors with Hadoop 2.7.4 or 3.0.0.
+1 findbugs 349 the patch passed
+1 javadoc 67 the patch passed
_ Other Tests _
+1 unit 227 hbase-client in the patch passed.
-1 unit 16638 hbase-server in the patch failed.
+1 asflicense 56 The patch does not generate ASF License warnings.
20291
Reason Tests
Failed junit tests hadoop.hbase.client.TestSnapshotTemporaryDirectoryWithRegionReplicas
hadoop.hbase.regionserver.TestEndToEndSplitTransaction
hadoop.hbase.quotas.TestSpaceQuotas
Subsystem Report/Notes
Docker Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-194/2/artifact/out/Dockerfile
GITHUB PR #194
Optional Tests dupname asflicense javac javadoc unit findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
uname Linux 77a82c3fe48e 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 17:16:02 UTC 2018 x86_64 GNU/Linux
Build tool maven
Personality /testptch/patchprocess/precommit/personality/provided.sh
git revision master / 962585d
maven version: Apache Maven 3.5.4 (1edded0938998edf8bf061f1ceb3cfdeccf443fe; 2018-06-17T18:33:14Z)
Default Java 1.8.0_181
findbugs v3.1.11
unit https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-194/2/artifact/out/patch-unit-hbase-server.txt
Test Results https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-194/2/testReport/
Max. process+thread count 5110 (vs. ulimit of 10000)
modules C: hbase-client hbase-server U: .
Console output https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-194/2/console
Powered by Apache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

@Apache9 Apache9 merged commit 4477dd5 into apache:master Apr 28, 2019
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.

3 participants