adoroszlai commented on PR #7572:
URL: https://github.com/apache/ozone/pull/7572#issuecomment-2542825890

   > I am adding a new argument for requests from S3 to let the server know 
where the requests are coming from and only check the requests from S3
   
   Instead of adding the flag, please use `ClientVersion`, which is already 
sent in requests.  Add a new constant and check for it in OM, like:
   
   
https://github.com/apache/ozone/blob/008f9a61f2965fa62979e493b547c6a744ad2459/hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/request/bucket/OMBucketCreateRequest.java#L474-L475
   
   > Do you think we need to add a test case in the xcompat tests for this?
   
   Yes, please.


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