[mutianf]

_{Stack created with GitHub Stacks CLI • Give Feedback 💬}

[gemini-code-assist]

Code Review

This pull request implements exponential backoff for retrying transport failures in RetryingVRpc and adds corresponding unit tests. The feedback highlights a potential NullPointerException if the operation deadline is null when calculating remaining time, suggesting a defensive check to default hasRemainingTime to true.

[gemini-code-assist]

If the operation does not have a deadline configured, context.getOperationInfo().getDeadline() may return null. Calling timeRemaining on a null deadline will throw a NullPointerException. We should defensively check if the deadline is null and default hasRemainingTime to true in that case.

Suggested change

	boolean hasRemainingTime =
	context.getOperationInfo().getDeadline().timeRemaining(TimeUnit.MILLISECONDS)
	- retryDelayMs
	> 1;
	io.grpc.Deadline deadline = context.getOperationInfo().getDeadline();
	boolean hasRemainingTime =
	deadline == null
	|| deadline.timeRemaining(TimeUnit.MILLISECONDS) - retryDelayMs > 1;