xiaguanglei commented on code in PR #6740:
URL: https://github.com/apache/hbase/pull/6740#discussion_r1991951073
##########
hbase-server/src/main/java/org/apache/hadoop/hbase/io/hfile/HFileBlock.java:
##########
@@ -1742,6 +1760,13 @@ protected HFileBlock
readBlockDataInternal(FSDataInputStream is, long offset,
onDiskSizeWithHeader = getOnDiskSizeWithHeader(headerBuf,
checksumSupport);
}
+ // if checksumType on headerBuf validate fail,the headerBuf may corrupted
Review Comment:
Yes, it needs to be expressed clearly here so that the meaning of this logic
can be easily understood,I have fixed the comment
--
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]