zymap opened a new pull request, #15710:
URL: https://github.com/apache/pulsar/pull/15710

   
   
   ### Motivation
   
   
   There have some cloud storages are compatible with S3
   APIs, such as aliyun-oss. Some other storages also use
   the S3 APIs and want to offload the data into them, but
   we only support the AWS or the Aliyun.
   The PR https://github.com/apache/pulsar/pull/8985 provides
   the Aliyun offload provider, but it has a force limitation of
   the `S3Constants.PROPERTY_S3_VIRTUAL_HOST_BUCKETS`. That
   is not a limitation on other storage services which compatible
   with S3 APIs.
   This PR provides  a more general offload provider `S3` which uses
   pure JClouds S3 metadata and allows people to override the
   default JClouds properties through system properties.
   
   
   ### Modifications
   
   - Add the pure S3 offload provider
   
   ### Verifying this change
   
   - [ ] Make sure that the change passes the CI checks.
   
   *(Please pick either of the following options)*
   
   This change is a trivial rework / code cleanup without any test coverage.
   
   *(or)*
   
   This change is already covered by existing tests, such as *(please describe 
tests)*.
   
   *(or)*
   
   This change added tests and can be verified as follows:
   
   *(example:)*
     - *Added integration tests for end-to-end deployment with large payloads 
(10MB)*
     - *Extended integration test for recovery after broker failure*
   
   ### Does this pull request potentially affect one of the following parts:
   
   *If `yes` was chosen, please highlight the changes*
   
     - Dependencies (does it add or upgrade a dependency): (yes / no)
     - The public API: (yes / no)
     - The schema: (yes / no / don't know)
     - The default values of configurations: (yes / no)
     - The wire protocol: (yes / no)
     - The rest endpoints: (yes / no)
     - The admin cli options: (yes / no)
     - Anything that affects deployment: (yes / no / don't know)
   
   ### Documentation
   
   Check the box below or label this PR directly.
   
   Need to update docs? 
   
   - [x] `doc-required` 
   (Your PR needs to update docs and you will update later)
     
   - [ ] `no-need-doc` 
   (Please explain why)
     
   - [ ] `doc` 
   (Your PR contains doc changes)
   
   - [ ] `doc-added`
   (Docs have been already added)


-- 
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: commits-unsubscr...@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to