Service Bus, EventHubs: review and fix retry policy usage as we onboard onto AMQP stack v2

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.3k stars 1.96k forks source link

Service Bus, EventHubs: review and fix retry policy usage as we onboard onto AMQP stack v2 #38733

Open lmolkova opened 7 months ago

lmolkova commented 7 months ago

Retries are currently handled on the public clients, but stack v2 will be responsible for the connection retries.

We need to test retries better and make sure we don't duplicate retries on different levels.

lmolkova commented 7 months ago

We need retries on:

sessions
links
anything else low-level involving networking

We should remove all the retry logic on the public surface and fully rely on retries for individual network operations. We should have tests that validate each low-level retries.