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.

Sign up for GitHub

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

Jump to bottom

Native multi-delete objects #38

Open
gaul opened this issue Jun 12, 2015 · 0 comments · May be fixed by #39
Open

Native multi-delete objects #38

gaul opened this issue Jun 12, 2015 · 0 comments · May be fixed by #39
Assignees
@timuralp

Comments

@gaul
Copy link
Member

gaul commented Jun 12, 2015

Presently SwiftProxy emulates bulk delete with multiple single object deletes. Instead it should issue a native bulk delete call via BlobStore.removeBlobs and fix up any cross-container issues with other logic.
@timuralp timuralp self-assigned this Jun 12, 2015
timuralp pushed a commit that referenced this issue Jun 13, 2015
Use removeBlobs() for bulk-delete.
c96cff5 
We should use removeBlobs() for bulk-delete. The change deletes blobs
en-masse. Further, we no longer will perform a HEAD on every blob, as
that's fairly expensive.

Fixes #38
@timuralp timuralp linked a pull request Jun 13, 2015 that will close this issue
Open
timuralp pushed a commit that referenced this issue Jun 15, 2015
Use removeBlobs() for bulk-delete.
52aa43b 
We should use removeBlobs() for bulk-delete. The change deletes blobs
en-masse. Further, we no longer will perform a HEAD on every blob, as
that's fairly expensive.

Fixes #38
timuralp pushed a commit that referenced this issue Jun 15, 2015
Use removeBlobs() for bulk-delete.
7c9fe56 
We should use removeBlobs() for bulk-delete. The change deletes blobs
en-masse. Further, we no longer will perform a HEAD on every blob, as
that's fairly expensive.

Fixes #38
timuralp pushed a commit that referenced this issue Jun 17, 2015
Use removeBlobs() for bulk-delete.
7ff978e 
We should use removeBlobs() for bulk-delete. The change deletes blobs
en-masse. Further, we no longer will perform a HEAD on every blob, as
that's fairly expensive.

Fixes #38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@timuralp timuralp

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Use removeBlobs() for bulk-delete. bouncestorage/swiftproxy
2 participants
@timuralp @gaul