Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implement Bulk Deletes for GCS Repository (#41368) #41681

Merged
merged 2 commits into from
Apr 30, 2019

Conversation

original-brownbear
Copy link
Member

  • Implement Bulk Deletes for GCS Repository

  • Just like Add Bulk Delete Api to BlobStore #40322 for AWS

  • We already had a bulk delete API but weren't using it from the blob container implementation, now we are using it

    • Made the bulk delete API also compliant with our interface that only suppresses errors about non existent blobs by stating failed deletes (I didn't use any bulk stat action here since having to stat here should be the exception anyway and it would make error handling a lot more complex)
  • Fixed bulk delete API to limit its batch size to 100 in line with GCS recommendations

back port of #41368

* Implement Bulk Deletes for GCS Repository

* Just like elastic#40322 for AWS
* We already had a bulk delete API but weren't using it from the blob container implementation, now we are using it
  * Made the bulk delete API also compliant with our interface that only suppresses errors about non existent blobs by stating failed deletes (I didn't use any bulk stat action here since having to stat here should be the exception anyway and it would make error handling a lot more complex)
* Fixed bulk delete API to limit its batch size to 100 in line with GCS recommendations
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-distributed

@original-brownbear
Copy link
Member Author

Jenkins run elasticsearch-ci/1

@original-brownbear original-brownbear merged commit 7cc4b9a into elastic:7.x Apr 30, 2019
@original-brownbear original-brownbear deleted the 41368-7.x branch April 30, 2019 15:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport :Distributed Coordination/Snapshot/Restore Anything directly related to the `_snapshot/*` APIs >enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants