Use removeBlobs() for bulk-delete.

[timuralp]

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

[gaul]

We really shouldn't have printStackTrace anywhere in our code.

[timuralp]

Done -- was following the other code in the project (e.g. AuthResource, ExceptionLogger, and IdentityResource).

[gaul]

Exception.printStackTrace does not go anywhere useful. If you want to log it, use logger.debug("some message", Exception) instead.

[gaul]

👍

[kahing]

👍

[gaul]

Do you want to use a Multimap instead?

[timuralp]

That would work too. I'll change the commit.

[gaul]

Do you need substring here?

[timuralp]

I think you're right, because we already do substring(1) on line 161. I'm not sure how this works right now for containers that start with "/"!

[gaul]

We are probably deleting a non-existent container which does not generate an error. We should enable/add some swift-tests to check for these behaviors.

[gaul]

@timuralp Do we we a path forward on this PR?

[timuralp]

@gaul I'll need to reload this into my brain. Will try to check it out over the weekend.