feat: Retry by default for uploads, blob deletes, metadata updates

[andrewsg]

No description provided.

[cojenco]

LGTM thanks for enabling retry by default woohoo

[andrewsg]

PTAL

[cojenco]

lgtm, thanks!

[ishowta]

@andrewsg I've noticed that other SDKs (at least Node.js and Go) seem to apply retries only to idempotent operations, as was previously the case in Python. If you don't mind, could you share a bit of insight into the rationale behind this change?

https://github.com/googleapis/nodejs-storage/blob/75c309c0761e4029dcd13024e748d8957052f766/src/storage.ts#L293

https://github.com/googleapis/google-cloud-go/blob/b1e3ce67d97275c7883e2b23617d070981d46bf7/storage/storage.go#L2355

[andrewsg]

@ishowta The C++ library currently has enabled retries by default on these operations. We are changing our policy around these specific operations to match the C++ library's precedent, starting with Python, but assuming there are no issues, later to include the other languages. We've determined that the potential negative effect of a double request for these listed operations is practically nil and the actual negative effect of having retries off by default is very large, as most people don't use generations or if_generation_match. However, if you do experience an unexpected issue with this change, please do let us know about it.

[ishowta]

Thank you for the explanation! In some of our projects, we've been configuring the Cloud Storage SDK to always retry, so I'll use this as a reference.