Re: [PR] HDDS-16127. Document OmKeyInfo key location version semantics [ozone]

["chihsuan (via GitHub)" <gi...@apache.org>]

chihsuan commented on code in PR #11018:
URL: https://github.com/apache/ozone/pull/11018#discussion_r3834917885


##########
hadoop-ozone/common/src/main/java/org/apache/hadoop/ozone/om/helpers/OmKeyInfo.java:
##########
@@ -74,6 +74,11 @@ public final class OmKeyInfo extends WithParentObjectId
   // name of key client specified
   private String keyName;
   private long dataSize;
+  /**
+   * Block locations of the key, one group per key version. No write path produces more than one group today, so a
+   * key holds a single version and {@code dataSize} covers all of its blocks. Object versioning (HDDS-15728) keeps
+   * each object version in its own OmKeyInfo rather than adding groups here.
+   */
   private List<OmKeyLocationInfoGroup> keyLocationVersions;

Review Comment:
   Thanks @smengcl I’ve updated the Javadoc and PR description accordingly. 🙏 



-- 
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