HDDS-2131. Optimize replication type and creation time calculation in S3 MPU list call.

[swagle]

What changes were proposed in this pull request?

Optimize listMultiPartUpload to not read from openKeyTable.

What is the link to the Apache JIRA

https://issues.apache.org/jira/browse/HDDS-2131

How was this patch tested?

Ran related unit tests.

[anuengineer]

+1. pending CI.

[swagle]

/retest

[bharatviswa504]

And also this is missing S3 end change to use the replication type and replication factor and show them in S3StorageType format.

Need to use this class for conversion. https://github.com/apache/hadoop-ozone/blob/master/hadoop-ozone/s3gateway/src/main/java/org/apache/hadoop/ozone/s3/util/S3StorageType.java

[swagle]

And also this is missing S3 end change to use the replication type and replication factor and show them in S3StorageType format.

Need to use this class for conversion. https://github.com/apache/hadoop-ozone/blob/master/hadoop-ozone/s3gateway/src/main/java/org/apache/hadoop/ozone/s3/util/S3StorageType.java

Thanks for pointing this one out, where does this change go?
Do you mean we should add S3StorageType to org.apache.hadoop.ozone.om.response.s3.multipart.S3InitiateMultipartUploadResponse#S3InitiateMultipartUploadResponse

[bharatviswa504]

And also this is missing S3 end change to use the replication type and replication factor and show them in S3StorageType format.
Need to use this class for conversion. https://github.com/apache/hadoop-ozone/blob/master/hadoop-ozone/s3gateway/src/main/java/org/apache/hadoop/ozone/s3/util/S3StorageType.java

Thanks for pointing this one out, where does this change go?
Do you mean we should add S3StorageType to org.apache.hadoop.ozone.om.response.s3.multipart.S3InitiateMultipartUploadResponse#S3InitiateMultipartUploadResponse

This was already taken care of. Sorry I thought this is missing.

[bharatviswa504]

+1 LGTM.
Can you confirm if test failures are not related to this patch.

[swagle]

Hi @bharatviswa504 the tests are failing since we do not have RATIS-707 fix, it is unrelated. Do you want me to rebase and push?

[bharatviswa504]

Thank You @swagle for the verification and for the contribution and @anuengineer for the review.