Skip to content

HBASE-23145 Remove out-of-date comments in StoreFlusher.java - #709

Merged
saintstack merged 1 commit into
apache:masterfrom
HanleyYang:HBASE-23145
Oct 11, 2019
Merged

HBASE-23145 Remove out-of-date comments in StoreFlusher.java#709
saintstack merged 1 commit into
apache:masterfrom
HanleyYang:HBASE-23145

Conversation

@HanleyYang

Copy link
Copy Markdown

No description provided.

@Apache-HBase

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
💙reexec2m 1sDocker mode activated.
_ Prechecks _
💚dupname0m 0sNo case conflicting files found.
💚hbaseanti0m 0sPatch does not have any anti-patterns.
💚@author0m 0sThe patch does not contain any @author tags.
💛test4tests0m 0sThe 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 _
💚mvninstall9m 8smaster passed
💚compile1m 15smaster passed
💚checkstyle1m 39smaster passed
💚shadedjars5m 53sbranch has no errors when building our shaded downstream artifacts.
💚javadoc0m 45smaster passed
💙spotbugs4m 53sUsed deprecated FindBugs config; considering switching to SpotBugs.
💚findbugs4m 50smaster passed
_ Patch Compile Tests _
💚mvninstall6m 40sthe patch passed
💚compile1m 8sthe patch passed
💚javac1m 8sthe patch passed
💚checkstyle1m 35sthe patch passed
💚whitespace0m 0sThe patch has no whitespace issues.
💚shadedjars5m 42spatch has no errors when building our shaded downstream artifacts.
💚hadoopcheck21m 6sPatch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.
💚javadoc0m 36sthe patch passed
💚findbugs4m 37sthe patch passed
_ Other Tests _
💔unit261m 58shbase-server in the patch failed.
💚asflicense0m 26sThe patch does not generate ASF License warnings.
336m 37s
ReasonTests
Failed junit testshadoop.hbase.namespace.TestNamespaceAuditor
hadoop.hbase.tool.TestSecureBulkLoadHFiles
hadoop.hbase.tool.TestBulkLoadHFiles
hadoop.hbase.client.TestAsyncTableGetMultiThreaded
SubsystemReport/Notes
DockerClient=19.03.3 Server=19.03.3 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-709/1/artifact/out/Dockerfile
GITHUB PR#709
Optional Testsdupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
unameLinux 60f2ed241d07 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-709/out/precommit/personality/provided.sh
git revisionmaster / ba12d5b
Default Java1.8.0_181
unithttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-709/1/artifact/out/patch-unit-hbase-server.txt
Test Resultshttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-709/1/testReport/
Max. process+thread count5022 (vs. ulimit of 10000)
modulesC: hbase-server U: hbase-server
Console outputhttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-709/1/console
versionsgit=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11
Powered byApache Yetus 0.11.0 https://yetus.apache.org

This message was automatically generated.

@saintstack
saintstack merged commit 473816d into apache:masterOct 11, 2019
Sign up for freeto 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.

4 participants

@HanleyYang@Apache-HBase@virajjasani@saintstack