kerneltime commented on a change in pull request #2542:
URL: https://github.com/apache/ozone/pull/2542#discussion_r690800501
##########
File path:
hadoop-hdds/container-service/src/main/java/org/apache/hadoop/ozone/container/keyvalue/KeyValueHandler.java
##########
@@ -595,6 +602,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
Review comment:
```suggestion
// Validate data only if the read chunk is issued by Ratis for its
internal logic.
// For client reads, the client is expected to validate.
```
--
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]
