Skip to content

Conversation

@ywelsch
Copy link
Contributor

@ywelsch ywelsch commented Apr 17, 2020

Adds ranged read support for GCS repositories in order to enable searchable snapshot support for GCS.

As part of this PR, I've extracted some of the test infrastructure to make sure that GoogleCloudStorageBlobContainerRetriesTests and S3BlobContainerRetriesTests are covering similar test (as I saw those diverging in what they cover)

@ywelsch ywelsch added >enhancement :Distributed Coordination/Snapshot/Restore Anything directly related to the `_snapshot/*` APIs v8.0.0 v7.8.0 labels Apr 17, 2020
@ywelsch ywelsch requested a review from tlrx April 17, 2020 14:34
Copy link
Member

@tlrx tlrx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM - Thanks for the clean up in tests 👍

@ywelsch ywelsch merged commit b6ab9b3 into elastic:master Apr 20, 2020
ywelsch added a commit that referenced this pull request Apr 20, 2020
Adds ranged read support for GCS repositories in order to enable searchable snapshot support
for GCS.

As part of this PR, I've extracted some of the test infrastructure to make sure that
GoogleCloudStorageBlobContainerRetriesTests and S3BlobContainerRetriesTests are covering
similar test (as I saw those diverging in what they cover)
@DaveCTurner DaveCTurner mentioned this pull request Apr 30, 2020
19 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants