xiaoyuyao commented on a change in pull request #1510: HDDS-2170. Add Object IDs and Update ID to Volume Object URL: https://github.com/apache/hadoop/pull/1510#discussion_r327411316
########## File path: hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/request/volume/OMVolumeCreateRequest.java ########## @@ -116,6 +116,11 @@ public OMClientResponse validateAndUpdateCache(OzoneManager ozoneManager, Collection<String> ozAdmins = ozoneManager.getOzoneAdmins(); try { omVolumeArgs = OmVolumeArgs.getFromProtobuf(volumeInfo); + // when you create a volume, we set both Object ID and update ID to the + // same ratis transaction ID. The Object ID will never change, but update + // ID will be set to transactionID each time we update the object. Review comment: Do we need to add handling for updateID change in some volume metadata operations such as set/add/remove ACL, addMetadata, etc ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org