avijayanhwx commented on a change in pull request #2538:
URL: https://github.com/apache/ozone/pull/2538#discussion_r689883026
##########
File path:
hadoop-hdds/container-service/src/main/java/org/apache/hadoop/ozone/container/keyvalue/KeyValueHandler.java
##########
@@ -595,6 +603,12 @@ ContainerCommandResponseProto handleReadChunk(
data = chunkManager.readChunk(kvContainer, blockID, chunkInfo,
dispatcherContext);
+ // Validate data only if the read chunk issued as a result of
+ // ratis call internally. For client reads, validation happens
+ // on the client
+ if (dispatcherContext.isReadFromTmpFile()) {
+ validateChunkData(data, chunkInfo);
Review comment:
Why do we have checksum verification in chunkManager.readChunk as well
as here?
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
