psalagnac commented on pull request #120: URL: https://github.com/apache/solr/pull/120#issuecomment-869802855
Hey @gerlowskija, Sorry for the late answer here. Thanks for you feedback. I think we reached an agreement to keep`BlobIndexInput` and remove the abstraction of the generic blob storage client. I will work with @athrog on following changes: - Rename `BlobBackupRepository` to `S3BackupRepository`, and update docs to target only S3. - Drop `BlobStorageClient` layer, and directly use AWS APIs in `S3BackupRepository` - As a second step, implement incremental test (this may came later depending on our bandwidth). -- 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: issues-unsubscr...@solr.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org For additional commands, e-mail: issues-h...@solr.apache.org
