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-22343 Make procedure retry interval configurable in test #215

Merged
merged 1 commit into from
May 4, 2019

Conversation

Apache9
Copy link
Contributor

@Apache9 Apache9 commented May 3, 2019

No description provided.

@Apache9 Apache9 requested a review from infraio May 3, 2019 14:26
Copy link
Contributor

@saintstack saintstack left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Excellent. I had an idea on how this should be done after reading first few lines and then I come to notice that you'd implemented what I was thinking only better and in a thorough way.

@Apache-HBase
Copy link

💔 -1 overall

Vote Subsystem Runtime Comment
0 reexec 23 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 2 new or modified test files.
_ master Compile Tests _
0 mvndep 23 Maven dependency ordering for branch
+1 mvninstall 233 master passed
+1 compile 71 master passed
+1 checkstyle 81 master passed
+1 shadedjars 262 branch has no errors when building our shaded downstream artifacts.
+1 findbugs 241 master passed
+1 javadoc 51 master passed
_ Patch Compile Tests _
0 mvndep 16 Maven dependency ordering for patch
+1 mvninstall 236 the patch passed
+1 compile 70 the patch passed
+1 javac 70 the patch passed
+1 checkstyle 85 the patch passed
+1 whitespace 0 The patch has no whitespace issues.
+1 shadedjars 261 patch has no errors when building our shaded downstream artifacts.
+1 hadoopcheck 479 Patch does not cause any errors with Hadoop 2.7.4 or 3.0.0.
+1 findbugs 226 the patch passed
+1 javadoc 48 the patch passed
_ Other Tests _
+1 unit 211 hbase-procedure in the patch passed.
-1 unit 8070 hbase-server in the patch failed.
+1 asflicense 54 The patch does not generate ASF License warnings.
10833
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-215/1/artifact/out/Dockerfile
GITHUB PR #215
Optional Tests dupname asflicense javac javadoc unit findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
uname Linux dcc668607c2c 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 / 0c0d595
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-215/1/artifact/out/patch-unit-hbase-server.txt
Test Results https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-215/1/testReport/
Max. process+thread count 4816 (vs. ulimit of 10000)
modules C: hbase-procedure hbase-server U: .
Console output https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-215/1/console
Powered by Apache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

@Apache9 Apache9 merged commit e884a25 into apache:master May 4, 2019
This was referenced Sep 14, 2024
This was referenced Oct 2, 2024
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