You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@hbase.apache.org by GitBox <gi...@apache.org> on 2021/05/11 22:41:47 UTC

[GitHub] [hbase] ndimiduk commented on pull request #3249: HBASE-25861 Correct the usage of Configuration#addDeprecation

ndimiduk commented on pull request #3249:
URL: https://github.com/apache/hbase/pull/3249#issuecomment-839245398


   > Thanks @ndimiduk and @saintstack for review. It's cleaner after using the configuration string values directly in the HBaseConfiguration#addDeprecatedKeys().
   
   Yes, this looks much better. It seems the test failure is related though, mind taking a look? If the jenkins test result screen doesn't include enough information for you, you can download all the [test logs](https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3249/2/artifact/yetus-jdk11-hadoop3-check/output/test_logs.zip).


-- 
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org