Particular / NServiceBus.Azure

Transports and storage for Windows Azure
https://docs.particular.net/nservicebus/azure/
Other
10 stars 15 forks source link

Poor exception handling in Cleanup #287

Closed SeanFeldman closed 7 years ago

SeanFeldman commented 7 years ago

Who's affected

Anyone using Azure DataBus and scaled out endpoints

Symptoms

Scaled out endpoint could attempt to remove blobs in parallel. This could result in instances listing blobs that are removed by another instance. When attempt is made to remove a blob that doesn't exist, exception is thrown and foreach loop will be interrupted, causing to skip the rest of the blobs that potentially could be deleted.

Workaround

Not required. Eventually blobs are removed by the last endpoint instance.

SeanFeldman commented 7 years ago

@yvesgoeleven please consider this for 6.2.7 as this is also a bug that should be addressed.