Azure / azure-sdk-for-java

This repository is for active development of the Azure SDK for Java. For consumers of the SDK we recommend visiting our public developer docs at https://docs.microsoft.com/java/azure/ or our versioned developer docs at https://azure.github.io/azure-sdk-for-java.
MIT License
2.35k stars 1.98k forks source link

[Tracking] Safety of cross regional retries on data plane writes that timeout on Gateway mode. #32554

Open NaluTripician opened 1 year ago

NaluTripician commented 1 year ago

A follow up to Issue #31367 and PR 32450

Investigation needed on whether data plane writes are safe to retry when they timeout on gateway mode.

ghost commented 1 year ago

Thanks for the feedback! We are routing this to the appropriate team for follow-up. cc @kushagraThapar, @TheovanKraay

ealsur commented 1 year ago

Revising the original design, Data Plane Writes are retryable if the HTTP timeout was on Connect / the request was never sent, not on Receive Timeouts.

If Java doesn't allow for distinction between Connect and Receive timeouts, probably safer to not retry.

kushagraThapar commented 8 months ago

@NaluTripician / @ealsur do we still need this issue? if not, can you please close it with the correct state, thanks!

ealsur commented 8 months ago

@kushagraThapar if the linked PR says that this is not needed then I guess it's not? @xinlian12 do you know?