rakeshadr commented on a change in pull request #2108:
URL: https://github.com/apache/ozone/pull/2108#discussion_r615689086
##########
File path:
hadoop-ozone/common/src/main/java/org/apache/hadoop/ozone/om/helpers/OmKeyInfo.java
##########
@@ -147,10 +151,31 @@ public void updateModifcationTime() {
*/
public void updateLocationInfoList(List<OmKeyLocationInfo> locationInfoList,
boolean isMpu) {
+ updateLocationInfoList(locationInfoList, isMpu, false);
+ }
+
+ /**
+ * updates the length of the each block in the list given.
+ * This will be called when the key is being committed to OzoneManager.
+ *
+ * @param locationInfoList list of locationInfo
Review comment:
Sure I will update javadoc. The purpose of `skipBlockIDCheck` third
param is to skip the blkLocation check in case of MPU multipart upload
overriding case, I have described it with an existing UT [in my below
comment](https://github.com/apache/ozone/pull/2108#discussion_r615686509).
--
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:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
