CSHARP-5712: withTransaction API retries too frequently

[sanych-sun]

Functionality is implemented. Spec changes assumes us to have prose test, it was not implemented yet.

[Copilot]

Pull request overview

This PR implements the RetryabilityHelper.GetRetryDelayMs method to control retry delays with exponential backoff, addressing the issue of transactions retrying too frequently. The implementation adds jitter to prevent thundering herd problems and includes comprehensive parameter validation.

Key Changes

• Adds GetRetryDelayMs method with exponential backoff calculation and random jitter
• Removes unused MongoDB.Driver.Core.Connections import from test file
• Adds comprehensive test coverage for valid inputs, edge cases, and parameter validation

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File	Description
src/MongoDB.Driver/Core/Operations/RetryabilityHelper.cs	Implements the core retry delay calculation with exponential backoff and jitter
tests/MongoDB.Driver.Tests/Core/Operations/RetryabilityHelperTests.cs	Adds unit tests for the new method and removes unused import

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

The calculation backoffInitial * Math.Pow(backoffBase, attempt - 1) may overflow for large attempt values, which would then be capped by Math.Min. Consider adding overflow protection or documenting the maximum safe attempt value to prevent unexpected behavior.

Suggested change

	return (int)(j * Math.Min(backoffMax, backoffInitial * Math.Pow(backoffBase, attempt - 1)));
	// compute the largest exponent such that backoffInitial * backoffBase^exponent <= backoffMax
	var maxExponent = Math.Log(backoffMax / (double)backoffInitial, backoffBase);
	var effectiveExponent = attempt - 1;
	double delayWithoutJitter;
	if (effectiveExponent >= maxExponent)
	{
	delayWithoutJitter = backoffMax;
	}
	else
	{
	delayWithoutJitter = backoffInitial * Math.Pow(backoffBase, effectiveExponent);
	}
	return (int)(j * delayWithoutJitter);

[Copilot]

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.