[GitHub] [ozone] umamaheswararao commented on a change in pull request #2709: HDDS-5816 Rearrange code and refactor some logic into new methods in prep for EC addition.

[GitBox <gi...@apache.org>]

umamaheswararao commented on a change in pull request #2709:
URL: https://github.com/apache/ozone/pull/2709#discussion_r722621445



##########
File path: hadoop-ozone/client/src/main/java/org/apache/hadoop/ozone/client/io/BlockOutputStreamEntry.java
##########
@@ -36,9 +36,16 @@
 import com.google.common.annotations.VisibleForTesting;
 
 /**
- * Helper class used inside {@link BlockOutputStream}.
+ * A BlockOutputStreamEntry manages the data writes into the DataNodes.
+ * It wraps BlockOutputStreams that are connecting to the DataNodes,
+ * and in the meantime accounts the length of data successfully written.
+ *
+ * The base implementation is handling Ratis-3 writes, with a single stream,
+ * but there can be other implementations that are using a different way, like
+ * EC writes where an entry manages the write into a block group via multiple

Review comment:
       Let's avoid to have EC related docs in master code.




-- 
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: issues-unsubscribe@ozone.apache.org

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



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org