Skip to content

HBASE-22922 Lock all regions to be merged in case of multi region merge - #542

Merged
saintstack merged 2 commits into
apache:masterfrom
stoty:HBASE-22922-master
Aug 27, 2019
Merged

HBASE-22922 Lock all regions to be merged in case of multi region merge#542
saintstack merged 2 commits into
apache:masterfrom
stoty:HBASE-22922-master

Conversation

@stoty

@stotystoty commented Aug 26, 2019

Copy link
Copy Markdown
Contributor

No description provided.

@Apache-HBase

Apache-HBase commented Aug 26, 2019

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
0reexec40Docker mode activated.
_ Prechecks _
+1dupname0No case conflicting files found.
+1hbaseanti0Patch does not have any anti-patterns.
+1@author0The patch does not contain any @author tags.
-0test4tests0The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
_ master Compile Tests _
+1mvninstall424master passed
+1compile73master passed
+1checkstyle110master passed
+1shadedjars361branch has no errors when building our shaded downstream artifacts.
+1javadoc48master passed
0spotbugs343Used deprecated FindBugs config; considering switching to SpotBugs.
+1findbugs339master passed
_ Patch Compile Tests _
+1mvninstall380the patch passed
+1compile60the patch passed
+1javac60the patch passed
+1checkstyle91the patch passed
+1whitespace0The patch has no whitespace issues.
+1shadedjars305patch has no errors when building our shaded downstream artifacts.
+1hadoopcheck1047Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.
+1javadoc36the patch passed
+1findbugs276the patch passed
_ Other Tests _
-1unit10262hbase-server in the patch failed.
+1asflicense26The patch does not generate ASF License warnings.
14327
ReasonTests
Failed junit testshadoop.hbase.coprocessor.TestMasterObserver
hadoop.hbase.master.assignment.TestMasterAbortWhileMergingTable
hadoop.hbase.master.procedure.TestIgnoreUnknownFamily
hadoop.hbase.TestSplitMerge
hadoop.hbase.client.TestSplitOrMergeAtTableLevel
hadoop.hbase.client.TestAdmin1
hadoop.hbase.master.TestMergeTableRegionsWhileRSCrash
hadoop.hbase.master.assignment.TestMergeTableRegionsProcedure
hadoop.hbase.TestSequenceIdMonotonicallyIncreasing
hadoop.hbase.replication.TestSerialReplication
hadoop.hbase.master.normalizer.TestSimpleRegionNormalizerOnCluster
hadoop.hbase.namespace.TestNamespaceAuditor
hadoop.hbase.client.TestTableFavoredNodes
hadoop.hbase.master.assignment.TestModifyTableWhileMerging
hadoop.hbase.client.TestAdmin2
hadoop.hbase.regionserver.TestRegionMergeTransactionOnCluster
hadoop.hbase.client.TestAsyncRegionAdminApi2
SubsystemReport/Notes
DockerClient=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-542/1/artifact/out/Dockerfile
GITHUB PR#542
Optional Testsdupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
unameLinux f7a8b811e0b8 4.15.0-54-generic #58-Ubuntu SMP Mon Jun 24 10:55:24 UTC 2019 x86_64 GNU/Linux
Build toolmaven
Personality/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-542/out/precommit/personality/provided.sh
git revisionmaster / 1ea5d8b
Default Java1.8.0_181
unithttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-542/1/artifact/out/patch-unit-hbase-server.txt
Test Resultshttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-542/1/testReport/
Max. process+thread count4358 (vs. ulimit of 10000)
modulesC: hbase-server U: hbase-server
Console outputhttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-542/1/console
versionsgit=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11
Powered byApache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Apache-HBase commented Aug 26, 2019

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
0reexec35Docker mode activated.
_ Prechecks _
+1dupname0No case conflicting files found.
+1hbaseanti0Patch does not have any anti-patterns.
+1@author0The patch does not contain any @author tags.
-0test4tests0The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
_ master Compile Tests _
+1mvninstall326master passed
+1compile56master passed
+1checkstyle79master passed
+1shadedjars271branch has no errors when building our shaded downstream artifacts.
+1javadoc38master passed
0spotbugs235Used deprecated FindBugs config; considering switching to SpotBugs.
+1findbugs233master passed
_ Patch Compile Tests _
+1mvninstall295the patch passed
+1compile56the patch passed
+1javac56the patch passed
+1checkstyle77the patch passed
+1whitespace1The patch has no whitespace issues.
+1shadedjars267patch has no errors when building our shaded downstream artifacts.
+1hadoopcheck919Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.
+1javadoc36the patch passed
+1findbugs247the patch passed
_ Other Tests _
+1unit8957hbase-server in the patch passed.
+1asflicense32The patch does not generate ASF License warnings.
12292
SubsystemReport/Notes
DockerClient=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-542/2/artifact/out/Dockerfile
GITHUB PR#542
Optional Testsdupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
unameLinux e667dc5737fe 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 GNU/Linux
Build toolmaven
Personality/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-542/out/precommit/personality/provided.sh
git revisionmaster / 1ea5d8b
Default Java1.8.0_181
Test Resultshttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-542/2/testReport/
Max. process+thread count4823 (vs. ulimit of 10000)
modulesC: hbase-server U: hbase-server
Console outputhttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-542/2/console
versionsgit=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11
Powered byApache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Apache-HBase commented Aug 27, 2019

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
0reexec121Docker mode activated.
_ Prechecks _
+1dupname0No case conflicting files found.
+1hbaseanti0Patch does not have any anti-patterns.
+1@author0The patch does not contain any @author tags.
-0test4tests0The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
_ master Compile Tests _
+1mvninstall446master passed
+1compile78master passed
+1checkstyle110master passed
+1shadedjars350branch has no errors when building our shaded downstream artifacts.
+1javadoc46master passed
0spotbugs274Used deprecated FindBugs config; considering switching to SpotBugs.
+1findbugs272master passed
_ Patch Compile Tests _
+1mvninstall428the patch passed
+1compile73the patch passed
+1javac73the patch passed
+1checkstyle92the patch passed
+1whitespace0The patch has no whitespace issues.
+1shadedjars342patch has no errors when building our shaded downstream artifacts.
+1hadoopcheck1355Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.
+1javadoc45the patch passed
+1findbugs301the patch passed
_ Other Tests _
-1unit17032hbase-server in the patch failed.
+1asflicense29The patch does not generate ASF License warnings.
21660
ReasonTests
Failed junit testshadoop.hbase.master.procedure.TestSCPWithReplicasWithoutZKCoordinated
hadoop.hbase.master.TestMasterShutdown
SubsystemReport/Notes
DockerClient=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-542/3/artifact/out/Dockerfile
GITHUB PR#542
Optional Testsdupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
unameLinux d32d80b0418e 4.15.0-48-generic #51-Ubuntu SMP Wed Apr 3 08:28:49 UTC 2019 x86_64 GNU/Linux
Build toolmaven
Personality/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-542/out/precommit/personality/provided.sh
git revisionmaster / 0438fdc
Default Java1.8.0_181
unithttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-542/3/artifact/out/patch-unit-hbase-server.txt
Test Resultshttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-542/3/testReport/
Max. process+thread count4654 (vs. ulimit of 10000)
modulesC: hbase-server U: hbase-server
Console outputhttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-542/3/console
versionsgit=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11
Powered byApache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

@saintstack
saintstack merged commit c882672 into apache:masterAug 27, 2019
asfgit pushed a commit that referenced this pull request Aug 27, 2019
@stoty@saintstack
HBASE-22922Lock all regions to be merged in case of multi region mer…
caba5cf
…ge (#542)
* HBASE-22922 Lock all regions to be merged in case of multi region merge
* HBASE-22922 Lock all regions to be merged in case of multi region merge
(addendum)
fix off-by-one error in patch
Signed-off-by: stack <stack@apache.org>
asfgit pushed a commit that referenced this pull request Aug 27, 2019
@stoty@saintstack
HBASE-22922Lock all regions to be merged in case of multi region mer…
76e9b84
…ge (#542)
* HBASE-22922 Lock all regions to be merged in case of multi region merge
* HBASE-22922 Lock all regions to be merged in case of multi region merge
(addendum)
fix off-by-one error in patch
Signed-off-by: stack <stack@apache.org>
asfgit pushed a commit that referenced this pull request Aug 27, 2019
@stoty@saintstack
HBASE-22922Lock all regions to be merged in case of multi region mer…
b0e8472
…ge (#542)
* HBASE-22922 Lock all regions to be merged in case of multi region merge
* HBASE-22922 Lock all regions to be merged in case of multi region merge
(addendum)
fix off-by-one error in patch
Signed-off-by: stack <stack@apache.org>
infraio pushed a commit to infraio/hbase that referenced this pull request Aug 17, 2020
@stotyGuanghao Zhang
HBASE-22922 Lock all regions to be merged in case of multi region mer…
1dd4bd4
…ge (apache#542)
* HBASE-22922 Lock all regions to be merged in case of multi region merge
* HBASE-22922 Lock all regions to be merged in case of multi region merge
(addendum)
fix off-by-one error in patch
Signed-off-by: stack <stack@apache.org>
symat pushed a commit to symat/hbase that referenced this pull request Feb 17, 2021
@stotyJenkins
HBASE-22922 Lock all regions to be merged in case of multi region mer…
a2b2ebf
…ge (apache#542)
* HBASE-22922 Lock all regions to be merged in case of multi region merge
* HBASE-22922 Lock all regions to be merged in case of multi region merge
(addendum)
fix off-by-one error in patch
Signed-off-by: stack <stack@apache.org>
(cherry picked from commit 76e9b84)
Change-Id: I765220d02f18a55c7f62b196a916e2ad9b24c348
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@stoty@Apache-HBase@saintstack