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

   ## What changes were proposed in this pull request?
   
   Fix duplicate initialization in S3 Gateway endpoints.
   
   1. `init()` is called from `initialization()`.  Both methods are 
`@PostConstruct`.  Thus `init()` is called twice.
       - remove `@PostConstruct` from `init()`
       - reduce visibility
       - add default no-op implementation
       - remove empty implementations from subclasses
   2. `BucketEndpoint` calls `BucketOperationHandler#initialization`, which 
creates duplicate `S3Auth` and `RequestParameters` instances
      - reuse existing instances in `copyDependenciesTo`
      - call `init()` instead of `initialization()`
      - add helper method `addHandler`
   
   https://issues.apache.org/jira/browse/HDDS-14399
   
   ## How was this patch tested?
   
   CI:
   https://github.com/adoroszlai/ozone/actions/runs/20852559826


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