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

   ## What changes were proposed in this pull request?
   
   Call `ClientProtocol#listParts` directly in `MultipartKeyHandler`. The 
`OzoneBucket` wrapper only forwards the volume and bucket names and listing 
parameters, so fetching it adds an unnecessary `InfoBucket` RPC when no 
expected bucket owner is specified. Owner verification remains in 
`AuditingObjectOperationHandler`.
   
   ## What is the link to the Apache JIRA
   
   https://issues.apache.org/jira/browse/HDDS-16636
   
   ## How was this patch tested?
   
   - Added tests for bucket lookup counts, owner checks, pagination and error 
mapping. Without the production change, 6 of the 11 `TestListParts` cases fail 
on extra bucket lookups. With the change, all 32 tests in `TestListParts`, 
`TestPermissionCheck` and `TestAbortMultipartUpload` pass.
   - [Fork CI 
passed](https://github.com/rich7420/ozone/actions/runs/36472011020). The 
Kubernetes job passed on rerun after an initial timeout waiting for SCM to exit 
safe mode.
   - [10×10 flaky-test-check 
passed](https://github.com/rich7420/ozone/actions/runs/36472051068) for 
`TestListParts` on Java 21.
   


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