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.

Sign up for GitHub

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

Jump to bottom

HBASE-22933 Do not need to kick reassign for rs group change any more #550

Merged
merged 1 commit into from
Aug 30, 2019
Merged

HBASE-22933 Do not need to kick reassign for rs group change any more #550

merged 1 commit into from
Aug 30, 2019

Conversation

Apache9
Copy link
Contributor

@Apache9 Apache9 commented Aug 27, 2019

No description provided.

@Apache9 Apache9 requested a review from infraio August 27, 2019 13:44
@Apache-HBase
Copy link

🎊 +1 overall

Vote Subsystem Runtime Comment
0 reexec 36 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.
-0 test4tests 0 The 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 _
+1 mvninstall 378 master passed
+1 compile 26 master passed
+1 checkstyle 15 master passed
+1 shadedjars 270 branch has no errors when building our shaded downstream artifacts.
+1 javadoc 24 master passed
0 spotbugs 48 Used deprecated FindBugs config; considering switching to SpotBugs.
+1 findbugs 45 master passed
_ Patch Compile Tests _
+1 mvninstall 298 the patch passed
+1 compile 26 the patch passed
+1 javac 26 the patch passed
+1 checkstyle 13 the patch passed
+1 whitespace 0 The patch has no whitespace issues.
+1 shadedjars 271 patch has no errors when building our shaded downstream artifacts.
+1 hadoopcheck 923 Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.
+1 javadoc 22 the patch passed
+1 findbugs 51 the patch passed
_ Other Tests _
+1 unit 285 hbase-rsgroup in the patch passed.
+1 asflicense 14 The patch does not generate ASF License warnings.
3063
Subsystem Report/Notes
Docker Client=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-550/1/artifact/out/Dockerfile
GITHUB PR #550
Optional Tests dupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
uname Linux 7111d18c5ca9 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-550/out/precommit/personality/provided.sh
git revision master / 5106f28
Default Java 1.8.0_181
Test Results https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-550/1/testReport/
Max. process+thread count 4526 (vs. ulimit of 10000)
modules C: hbase-rsgroup U: hbase-rsgroup
Console output https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-550/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 200 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.
-0 test4tests 0 The 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 _
+1 mvninstall 494 master passed
+1 compile 36 master passed
+1 checkstyle 18 master passed
+1 shadedjars 399 branch has no errors when building our shaded downstream artifacts.
+1 javadoc 29 master passed
0 spotbugs 76 Used deprecated FindBugs config; considering switching to SpotBugs.
+1 findbugs 73 master passed
_ Patch Compile Tests _
+1 mvninstall 475 the patch passed
+1 compile 36 the patch passed
+1 javac 36 the patch passed
+1 checkstyle 15 the patch passed
+1 whitespace 0 The patch has no whitespace issues.
+1 shadedjars 390 patch has no errors when building our shaded downstream artifacts.
+1 hadoopcheck 1418 Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.
+1 javadoc 27 the patch passed
+1 findbugs 73 the patch passed
_ Other Tests _
+1 unit 816 hbase-rsgroup in the patch passed.
+1 asflicense 16 The patch does not generate ASF License warnings.
5072
Subsystem Report/Notes
Docker Client=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-550/2/artifact/out/Dockerfile
GITHUB PR #550
Optional Tests dupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
uname Linux d528c70b0387 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-550/out/precommit/personality/provided.sh
git revision master / 56980fb
Default Java 1.8.0_181
Test Results https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-550/2/testReport/
Max. process+thread count 4324 (vs. ulimit of 10000)
modules C: hbase-rsgroup U: hbase-rsgroup
Console output https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-550/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 39 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.
-0 test4tests 0 The 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 _
+1 mvninstall 330 master passed
+1 compile 27 master passed
+1 checkstyle 16 master passed
+1 shadedjars 280 branch has no errors when building our shaded downstream artifacts.
+1 javadoc 23 master passed
0 spotbugs 48 Used deprecated FindBugs config; considering switching to SpotBugs.
+1 findbugs 47 master passed
_ Patch Compile Tests _
+1 mvninstall 305 the patch passed
+1 compile 26 the patch passed
+1 javac 26 the patch passed
+1 checkstyle 13 the patch passed
+1 whitespace 0 The patch has no whitespace issues.
+1 shadedjars 290 patch has no errors when building our shaded downstream artifacts.
+1 hadoopcheck 952 Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.
+1 javadoc 21 the patch passed
+1 findbugs 51 the patch passed
_ Other Tests _
+1 unit 267 hbase-rsgroup in the patch passed.
+1 asflicense 13 The patch does not generate ASF License warnings.
3068
Subsystem Report/Notes
Docker Client=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-550/3/artifact/out/Dockerfile
GITHUB PR #550
Optional Tests dupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
uname Linux 89bd400bc974 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-550/out/precommit/personality/provided.sh
git revision master / 56980fb
Default Java 1.8.0_181
Test Results https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-550/3/testReport/
Max. process+thread count 4591 (vs. ulimit of 10000)
modules C: hbase-rsgroup U: hbase-rsgroup
Console output https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-550/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.

@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.
-0 test4tests 0 The 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 _
+1 mvninstall 391 master passed
+1 compile 26 master passed
+1 checkstyle 15 master passed
+1 shadedjars 277 branch has no errors when building our shaded downstream artifacts.
+1 javadoc 23 master passed
0 spotbugs 53 Used deprecated FindBugs config; considering switching to SpotBugs.
+1 findbugs 51 master passed
_ Patch Compile Tests _
+1 mvninstall 297 the patch passed
+1 compile 25 the patch passed
+1 javac 25 the patch passed
+1 checkstyle 13 the patch passed
+1 whitespace 0 The patch has no whitespace issues.
+1 shadedjars 272 patch has no errors when building our shaded downstream artifacts.
+1 hadoopcheck 984 Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.
+1 javadoc 25 the patch passed
+1 findbugs 65 the patch passed
_ Other Tests _
+1 unit 303 hbase-rsgroup in the patch passed.
+1 asflicense 13 The patch does not generate ASF License warnings.
3294
Subsystem Report/Notes
Docker Client=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-550/4/artifact/out/Dockerfile
GITHUB PR #550
Optional Tests dupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
uname Linux 897a9e17f28d 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-550/out/precommit/personality/provided.sh
git revision master / 83e7794
Default Java 1.8.0_181
Test Results https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-550/4/testReport/
Max. process+thread count 4500 (vs. ulimit of 10000)
modules C: hbase-rsgroup U: hbase-rsgroup
Console output https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-550/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.

@Apache-HBase
Copy link

🎊 +1 overall

Vote Subsystem Runtime Comment
0 reexec 71 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.
-0 test4tests 0 The 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 _
+1 mvninstall 358 master passed
+1 compile 24 master passed
+1 checkstyle 13 master passed
+1 shadedjars 300 branch has no errors when building our shaded downstream artifacts.
+1 javadoc 22 master passed
0 spotbugs 49 Used deprecated FindBugs config; considering switching to SpotBugs.
+1 findbugs 47 master passed
_ Patch Compile Tests _
+1 mvninstall 330 the patch passed
+1 compile 27 the patch passed
+1 javac 27 the patch passed
+1 checkstyle 12 the patch passed
+1 whitespace 0 The patch has no whitespace issues.
+1 shadedjars 300 patch has no errors when building our shaded downstream artifacts.
+1 hadoopcheck 1027 Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.
+1 javadoc 19 the patch passed
+1 findbugs 52 the patch passed
_ Other Tests _
+1 unit 439 hbase-rsgroup in the patch passed.
+1 asflicense 13 The patch does not generate ASF License warnings.
3447
Subsystem Report/Notes
Docker Client=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-550/5/artifact/out/Dockerfile
GITHUB PR #550
Optional Tests dupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
uname Linux cdfd3c2e8617 4.15.0-54-generic #58-Ubuntu SMP Mon Jun 24 10:55:24 UTC 2019 x86_64 GNU/Linux
Build tool maven
Personality /home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-550/out/precommit/personality/provided.sh
git revision master / f6a4c66
Default Java 1.8.0_181
Test Results https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-550/5/testReport/
Max. process+thread count 4038 (vs. ulimit of 10000)
modules C: hbase-rsgroup U: hbase-rsgroup
Console output https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-550/5/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.

@Apache9 Apache9 merged commit 090c55f into apache:master Aug 30, 2019
asfgit pushed a commit that referenced this pull request Aug 30, 2019
@Apache9
HBASE-22933 Do not need to kick reassign for rs group change any more (…
4a4208f 
…#550)

Signed-off-by: Guanghao Zhang <[email protected]>
asfgit pushed a commit that referenced this pull request Aug 30, 2019
@Apache9
HBASE-22933 Do not need to kick reassign for rs group change any more (…
1580507 
…#550)

Signed-off-by: Guanghao Zhang <[email protected]>
infraio pushed a commit to infraio/hbase that referenced this pull request Aug 17, 2020
@Apache9
HBASE-22933 Do not need to kick reassign for rs group change any more (…
92f9467 
…apache#550)

Signed-off-by: Guanghao Zhang <[email protected]>
symat pushed a commit to symat/hbase that referenced this pull request Feb 17, 2021
@Apache9
HBASE-22933 Do not need to kick reassign for rs group change any more (…
7152a40 
…apache#550)

Signed-off-by: Guanghao Zhang <[email protected]>
(cherry picked from commit 1580507)

Change-Id: I95ef4761e9352c3e26c071ec96fce35101eba516
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@infraio infraio infraio approved these changes

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
@Apache9 @Apache-HBase @infraio