Don't use 0 milliseconds as sending retry interval.

This commit is contained in:
Nicholas Tinsley
2024-02-20 13:51:10 -05:00
committed by Greyson Parrelli
parent 6bc41776b1
commit 9dc856202a

View File

@@ -492,7 +492,11 @@ public final class PushGroupSendJob extends PushSendJob {
} else if (!networkFailures.isEmpty()) {
long retryAfter = results.stream()
.filter(r -> r.getRateLimitFailure() != null)
.map(r -> r.getRateLimitFailure().getRetryAfterMilliseconds().orElse(-1L))
.map(r -> {
long milliseconds = r.getRateLimitFailure().getRetryAfterMilliseconds().orElse(-1L);
return (milliseconds > 0) ? milliseconds : -1L;
}
)
.max(Long::compare)
.orElse(-1L);
Log.w(TAG, "Retrying because there were " + networkFailures.size() + " network failures. retryAfter: " + retryAfter);