From 4ce4491749b23309bd868fdb358f649e40b1a52b Mon Sep 17 00:00:00 2001 From: Dongjoon Hyun Date: Tue, 2 May 2023 12:04:53 -0700 Subject: [PATCH 1/2] HADOOP-18727. Fix WriteOperations.listMultipartUploads function description --- .../java/org/apache/hadoop/fs/s3a/WriteOperations.java | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/hadoop-tools/hadoop-aws/src/main/java/org/apache/hadoop/fs/s3a/WriteOperations.java b/hadoop-tools/hadoop-aws/src/main/java/org/apache/hadoop/fs/s3a/WriteOperations.java index bd6e5ac14731f..48983641f9349 100644 --- a/hadoop-tools/hadoop-aws/src/main/java/org/apache/hadoop/fs/s3a/WriteOperations.java +++ b/hadoop-tools/hadoop-aws/src/main/java/org/apache/hadoop/fs/s3a/WriteOperations.java @@ -192,11 +192,10 @@ int abortMultipartUploadsUnderPath(String prefix) throws IOException; /** - * Abort multipart uploads under a path: limited to the first - * few hundred. - * @param prefix prefix for uploads to abort - * @return a count of aborts - * @throws IOException trouble; FileNotFoundExceptions are swallowed. + * List in-progress multipart uploads under a path + * @param prefix prefix for uploads to list + * @return a list of in-progress multipart uploads + * @throws IOException on problems */ List listMultipartUploads(String prefix) throws IOException; From f44210e8270ebfb8e187093c4cc17b431baaef66 Mon Sep 17 00:00:00 2001 From: Dongjoon Hyun Date: Wed, 3 May 2023 16:41:22 -0700 Subject: [PATCH 2/2] Address comments --- .../main/java/org/apache/hadoop/fs/s3a/WriteOperations.java | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/hadoop-tools/hadoop-aws/src/main/java/org/apache/hadoop/fs/s3a/WriteOperations.java b/hadoop-tools/hadoop-aws/src/main/java/org/apache/hadoop/fs/s3a/WriteOperations.java index 48983641f9349..1c3d368857583 100644 --- a/hadoop-tools/hadoop-aws/src/main/java/org/apache/hadoop/fs/s3a/WriteOperations.java +++ b/hadoop-tools/hadoop-aws/src/main/java/org/apache/hadoop/fs/s3a/WriteOperations.java @@ -192,7 +192,8 @@ int abortMultipartUploadsUnderPath(String prefix) throws IOException; /** - * List in-progress multipart uploads under a path + * List in-progress multipart uploads under a path: limited to the first + * few hundred. * @param prefix prefix for uploads to list * @return a list of in-progress multipart uploads * @throws IOException on problems