swagle commented on a change in pull request #50: HDDS-2131. Optimize replication type and creation time calculation in S3 MPU list call. URL: https://github.com/apache/hadoop-ozone/pull/50#discussion_r337670962
########## File path: hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/KeyManagerImpl.java ########## @@ -893,17 +893,18 @@ private OmMultipartInfo createMultipartInfo(OmKeyArgs keyArgs, // Not checking if there is an already key for this in the keyTable, as // during final complete multipart upload we take care of this. - + long currentTime = Time.now(); Map<Integer, PartKeyInfo> partKeyInfoMap = new HashMap<>(); OmMultipartKeyInfo multipartKeyInfo = new OmMultipartKeyInfo( - multipartUploadID, partKeyInfoMap); + multipartUploadID, currentTime, keyArgs.getType(), Review comment: Hi @bharatviswa504, I had already changed this in my previous patch. Since it is the only constructor code would not compile otherwise. ---------------------------------------------------------------- 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: hdfs-dev-unsubscr...@hadoop.apache.org For additional commands, e-mail: hdfs-dev-h...@hadoop.apache.org