Skip to content

HBASE-22047 LeaseException in Scan should be retried - #158

Merged
saintstack merged 1 commit into
apache:masterfrom
hextriclosan:HBASE-22047
Apr 23, 2019
Merged

HBASE-22047 LeaseException in Scan should be retried#158
saintstack merged 1 commit into
apache:masterfrom
hextriclosan:HBASE-22047

Conversation

@hextriclosan

@hextriclosanhextriclosan commented Apr 16, 2019

Copy link
Copy Markdown
Contributor

No description provided.

@Apache-HBase

Apache-HBase commented Apr 16, 2019

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
0reexec46Docker mode activated.
_ Prechecks _
+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 _
+1mvninstall245master passed
+1compile25master passed
+1checkstyle33master passed
+1shadedjars270branch has no errors when building our shaded downstream artifacts.
+1findbugs59master passed
+1javadoc23master passed
_ Patch Compile Tests _
+1mvninstall244the patch passed
+1compile24the patch passed
+1javac24the patch passed
+1checkstyle29the patch passed
+1whitespace0The patch has no whitespace issues.
+1shadedjars264patch has no errors when building our shaded downstream artifacts.
+1hadoopcheck481Patch does not cause any errors with Hadoop 2.7.4 or 3.0.0.
+1findbugs68the patch passed
+1javadoc21the patch passed
_ Other Tests _
+1unit187hbase-client in the patch passed.
+1asflicense10The patch does not generate ASF License warnings.
2101
SubsystemReport/Notes
DockerClient=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-158/1/artifact/out/Dockerfile
GITHUB PR#158
Optional Testsdupname asflicense javac javadoc unit findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
unameLinux 4b9909d106ea 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 / 20f72f5
mavenversion: Apache Maven 3.5.4 (1edded0938998edf8bf061f1ceb3cfdeccf443fe; 2018-06-17T18:33:14Z)
Default Java1.8.0_181
findbugsv3.1.11
Test Resultshttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-158/1/testReport/
Max. process+thread count312 (vs. ulimit of 10000)
modulesC: hbase-client U: hbase-client
Console outputhttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-158/1/console
Powered byApache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Apache-HBase commented Apr 20, 2019

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
0reexec29Docker mode activated.
_ Prechecks _
+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 _
+1mvninstall264master passed
+1compile24master passed
+1checkstyle30master passed
+1shadedjars275branch has no errors when building our shaded downstream artifacts.
+1findbugs64master passed
+1javadoc24master passed
_ Patch Compile Tests _
+1mvninstall250the patch passed
+1compile27the patch passed
+1javac27the patch passed
+1checkstyle30the patch passed
+1whitespace0The patch has no whitespace issues.
+1shadedjars277patch has no errors when building our shaded downstream artifacts.
+1hadoopcheck541Patch does not cause any errors with Hadoop 2.7.4 or 3.0.0.
+1findbugs70the patch passed
+1javadoc22the patch passed
_ Other Tests _
+1unit196hbase-client in the patch passed.
+1asflicense11The patch does not generate ASF License warnings.
2189
SubsystemReport/Notes
DockerClient=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-158/2/artifact/out/Dockerfile
GITHUB PR#158
Optional Testsdupname asflicense javac javadoc unit findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
unameLinux 20814df71685 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 / f30d6c9
mavenversion: Apache Maven 3.5.4 (1edded0938998edf8bf061f1ceb3cfdeccf443fe; 2018-06-17T18:33:14Z)
Default Java1.8.0_181
findbugsv3.1.11
Test Resultshttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-158/2/testReport/
Max. process+thread count302 (vs. ulimit of 10000)
modulesC: hbase-client U: hbase-client
Console outputhttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-158/2/console
Powered byApache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

@saintstack

saintstack commented Apr 23, 2019

Copy link
Copy Markdown
Contributor

Change looks good to me. What you think @allan163

@allan163

allan163 commented Apr 23, 2019

Copy link
Copy Markdown
Member

+1 for the patch

@saintstack
saintstack merged commit da286b9 into apache:masterApr 23, 2019
@saintstack

saintstack commented Apr 23, 2019

Copy link
Copy Markdown
Contributor

Pushed to branch-2.1+ and to master.

stoty pushed a commit to stoty/hbase that referenced this pull request Nov 26, 2024
@BecseiL@bbeaudreaultGitHub Enterprise
CDPD-73638: Backport HBASE-27531 AsyncRequestFutureImpl unnecessarily…
c016f3f
… clears meta cache for full server (apache#4930) (apache#158)
* CDPD-73638: Backport HBASE-27531 AsyncRequestFutureImpl unnecessarily clears meta cache for full server (apache#4930)
Signed-off-by: Duo Zhang <zhangduo@apache.org>
(cherry picked from commit 1c98931)
Change-Id: If18b2add03e0509b7b778fcb14c2448353adf442
* CDPD-73638 - TestAsyncProcess.java fix
Change-Id: I1d18f40b052ca2b62bd5428678cbd6a90a859977
---------
Co-authored-by: Bryan Beaudreault <bbeaudreault@apache.org>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer
@lujiefsilujiefsilujiefsi approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@hextriclosan@Apache-HBase@saintstack@allan163@lujiefsi