[ 
https://issues.apache.org/jira/browse/HBASE-24234?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17090460#comment-17090460
 ] 

Wellington Chevreuil edited comment on HBASE-24234 at 4/23/20, 10:06 AM:
-------------------------------------------------------------------------

[~liuml07], [~xiaozhang], thanks for the heads up! I had just committed 
HBASE-23047 into branch-2.2. Let me know if anything else is pending here. 

Regards!


was (Author: wchevreuil):
[~liuml07], [~xiaozhang], thanks for heads up! I had just committed HBASE-23047 
into branch-2.2. Let me know if anything else is pending here. 

Regards!

> ChecksumUtil class validateChecksum method log level does not match
> -------------------------------------------------------------------
>
>                 Key: HBASE-24234
>                 URL: https://issues.apache.org/jira/browse/HBASE-24234
>             Project: HBase
>          Issue Type: Bug
>          Components: HFile
>    Affects Versions: 2.2.0, 2.2.1, 2.2.2, 2.2.3, 2.2.4
>            Reporter: Xiao Zhang
>            Assignee: Xiao Zhang
>            Priority: Minor
>
> In the validateChecksum method of the ChecksumUtil class, still use LOG.info 
> after judging LOG.isTraceEnabled.
> eg:
>  if (LOG.isTraceEnabled()) {
>  LOG.info("dataLength=" + buffer.capacity()
>  + ", sizeWithHeader=" + onDiskDataSizeWithHeader
>  + ", checksumType=" + cktype.getName()
>  + ", file=" + pathName
>  + ", offset=" + offset
>  + ", headerSize=" + hdrSize
>  + ", bytesPerChecksum=" + bytesPerChecksum);
>  }



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to