xiaoyuyao commented on a change in pull request #2108:
URL: https://github.com/apache/ozone/pull/2108#discussion_r611797401
##########
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:
NIT: can we add all the javadoc for the added params?
--
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]
