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-22294 Removed deprecated method from WALKeyImpl #186

Merged
merged 1 commit into from
Apr 24, 2019
Merged

HBASE-22294 Removed deprecated method from WALKeyImpl #186

merged 1 commit into from
Apr 24, 2019

Conversation

anis016
Copy link
Contributor

@anis016 anis016 commented Apr 24, 2019

Removed deprecated methods in ReplicationLoadSink.
Fixes: HBASE-22294

Copy link
Contributor

@HorizonNet HorizonNet left a comment

Choose a reason for hiding this comment

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

Conditional approval, pending QA bot.

@Apache-HBase
Copy link

🎊 +1 overall

Vote Subsystem Runtime Comment
0 reexec 20 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.
-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 269 master passed
+1 compile 52 master passed
+1 checkstyle 76 master passed
+1 shadedjars 279 branch has no errors when building our shaded downstream artifacts.
+1 findbugs 193 master passed
+1 javadoc 34 master passed
_ Patch Compile Tests _
+1 mvninstall 251 the patch passed
+1 compile 54 the patch passed
+1 javac 54 the patch passed
+1 checkstyle 74 the patch passed
+1 whitespace 0 The patch has no whitespace issues.
+1 shadedjars 278 patch has no errors when building our shaded downstream artifacts.
+1 hadoopcheck 538 Patch does not cause any errors with Hadoop 2.7.4 or 3.0.0.
+1 findbugs 193 the patch passed
+1 javadoc 32 the patch passed
_ Other Tests _
+1 unit 8604 hbase-server in the patch passed.
+1 asflicense 25 The patch does not generate ASF License warnings.
11044
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-186/1/artifact/out/Dockerfile
GITHUB PR #186
Optional Tests dupname asflicense javac javadoc unit findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
uname Linux 054187f2f0e7 4.4.0-143-generic #169~14.04.2-Ubuntu SMP Wed Feb 13 15:00:41 UTC 2019 x86_64 GNU/Linux
Build tool maven
Personality /testptch/patchprocess/precommit/personality/provided.sh
git revision master / ab3d6cf
maven version: Apache Maven 3.5.4 (1edded0938998edf8bf061f1ceb3cfdeccf443fe; 2018-06-17T18:33:14Z)
Default Java 1.8.0_181
findbugs v3.1.11
Test Results https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-186/1/testReport/
Max. process+thread count 4849 (vs. ulimit of 10000)
modules C: hbase-server U: hbase-server
Console output https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-186/1/console
Powered by Apache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

@HorizonNet HorizonNet merged commit 5bc7eda into apache:master Apr 24, 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