apurtell commented on pull request #4241:
URL: https://github.com/apache/hbase/pull/4241#issuecomment-1072631799


   Latest precommit result is good: 
   
   https://github.com/apache/hbase/pull/4241#issuecomment-1072071660
   
   There is an error prone result to address:
   
       [ERROR] Failed to execute goal 
org.apache.maven.plugins:maven-compiler-plugin:3.8.1:testCompile 
(default-testCompile) on project hbase-server: Compilation failure
       [ERROR] 
/home/jenkins/jenkins-home/workspace/Base-PreCommit-GitHub-PR_PR-4241@2/yetus-general-check/src/hbase-server/src/test/java/org/apache/hadoop/hbase/regionserver/TestMergesSplitsAddToTracker.java:[237,36]
 error: [ReturnValueIgnored] Return value of 'contains' must be used
       [ERROR]     (see https://errorprone.info/bugpattern/ReturnValueIgnored)
   
   
   Not strictly necessary to address javadoc or checkstyle issues in a 
backport, I think, but let me do it anyway. 
   
   I will also see if I can reproduce the TestHFileProcedurePrettyPrinter 
failure on the earlier precommit run. Perhaps it is flaky now. 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to