[JAMES-3318] DeletedMessageVault::deleteMessage should call directly DumbBlobStore::delete

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Done
• Fix Version/s: 3.6.0
• Component/s: Blob
• Assignee: Unassigned
• Reporter: Benoit Tellier
• Created: 24/Jul/20 02:44
• Updated: 28/Jul/20 03:29
• Resolved: 28/Jul/20 03:29

Description

== Why

DeletedMessage vault needs to delete related content. We need the deletion to take place immediately.

However appending into the vault is a common operation and we want to have metrics for it.

We want to keep the BlobStore API simple and easy to use.

== DOD

`DeDuplicationBlobStore::delete` is a noop.

All code for `BlobStore::delete` is moved into `PassThroughBlobStoreTest`.

DeletedMessage functionality of the DeletedMessageVault keeps unaffected.

== How

Implement `DeletedMessageVault::deleteMessage` via direct access to the DumbBlobStore.