ChenSammi commented on a change in pull request #1701:
URL: https://github.com/apache/ozone/pull/1701#discussion_r565900409



##########
File path: 
hadoop-ozone/s3gateway/src/main/java/org/apache/hadoop/ozone/s3/endpoint/BucketEndpoint.java
##########
@@ -75,7 +87,7 @@
   @GET
   @SuppressFBWarnings
   @SuppressWarnings("parameternumber")
-  public Response list(
+  public Response get(

Review comment:
       The annotation @GET defines that this get or list function is called 
when HTTP GET on bucket is called.  
   There are plenty of HTTP GET operations on bucket, such as list object, get 
ACL, each with different parameters. 
   
   Before this patch, the GET operation on the bucket only supports bucket 
list. With this patch, get ACL and list bucket are both supported.  In the 
future, when we need to support another GET operation, we just need to add more 
logic to this bucket endpoint get function. 
     




----------------------------------------------------------------
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]

Reply via email to