Skip to content

JAMES-3150 Better handle massive deletions as part of BloomFilterGCAl… - #665

Merged
chibenwa merged 1 commit into
apache:masterfrom
chibenwa:JAMES-3150-delete-perfs
Sep 23, 2021
Merged

JAMES-3150 Better handle massive deletions as part of BloomFilterGCAl…#665
chibenwa merged 1 commit into
apache:masterfrom
chibenwa:JAMES-3150-delete-perfs

Conversation

@chibenwa

Copy link
Copy Markdown
Contributor

…gorithm

Before we were deleting 128 objjects in parrallel.

This approach proved to be slow: we took 25 hours for 35M deletions...

With this proposed changeset we delete 1000 blobs per requests,
16 in parallel, using a S3 API to delete several objects at
once.

…gorithm

Before we were deleting 128 objjects in parrallel.

This approach proved to be slow: we took 25 hours for 35M deletions...

With this proposed changeset we delete 1000 blobs per requests,
16 in parallel, using a S3 API to delete several objects at
once.
@chibenwa chibenwa self-assigned this Sep 21, 2021

@vttranlina vttranlina left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

With AWS S3, they limit the request rate (throw an exception if too much request in time).
Some other cloud vendors may be the same.
So, I have a bit of thinking here

@Arsnael

Arsnael commented Sep 22, 2021

Copy link
Copy Markdown
Contributor

With AWS S3, they limit the request rate (throw an exception if too much request in time).
Some other cloud vendors may be the same.
So, I have a bit of thinking here

Not sure about how it works exactly but I would expect the S3 java client being used having a retry mechanism for such issue? If not we would have a lot of blobs disappearing I guess not only there?

@chibenwa
chibenwa merged commit d527312 into apache:master Sep 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants