Storage.delete returns false on NotFound, BatchRequest.delete shows failure

[mziccard]

If we try to delete a non-existing blob storage.delete(nonExistingBlob) returns false.
If we try to delete a non-existing blob with a batch request:

BatchRequest batchRequest = BatchRequest.builder()
    .delete(nonExistingBlob)
    .build();
...
batchResponse.deletes().get(0).failed(); // This is true

The deletion of a non-existing-blob is considered a failure in BatchResponse.
Wouldn't it be better to make this two ways of deleting a blob exhibit the same behavior?

[aozarov]

Yes, it would. Do you mind to fix that?

Also, currently our javadoc says "@return true if blob was deleted", should we also indicate that false means "failed to delete or not found"?

[mziccard]

Sure I'll fix this!