adoroszlai opened a new pull request, #9525:
URL: https://github.com/apache/ozone/pull/9525

   ## What changes were proposed in this pull request?
   
   - Extract constants for `@QueryParam` values to `S3Consts.QueryParams`.
   - Extract constants for `@PathParam` values to the endpoint class, because 
these depend on the `@Path` from the same class.
   - Use constants from `HttpHeaders` for `@HeaderParam` values.  Some were 
already used as static imports, replace them with qualified access for clarity 
and consistency.
   - Attempt to clarify usage of `OzoneConsts.ETAG` vs. `HttpHeaders.ETAG` for 
metadata vs. header purposes.  BTW the two values are the same.
   
   https://issues.apache.org/jira/browse/HDDS-14208
   
   ## How was this patch tested?
   
   Ran all S3 unit tests locally.
   
   CI:
   https://github.com/adoroszlai/ozone/actions/runs/20345474287


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

To unsubscribe, e-mail: [email protected]

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