I am open to suggestions on how to split these up, but these changes are tightly coupled. I could remove the Glacier offloader, but that would be just one class.
The refactor moved all of the logic for getting the credentials out of a single class with multiple if/the/else if/else if constructs and into a factory. Then I refactored the S3, and GCS to use the new factory methods, and added Azure. [ Full content available at: https://github.com/apache/pulsar/pull/2615 ] This message was relayed via gitbox.apache.org for [email protected]
