From f42936cabd28668cc98115ce45752e7f98982ca8 Mon Sep 17 00:00:00 2001 From: Kuan Wee Loong Date: Thu, 18 Apr 2024 19:25:33 +0800 Subject: [PATCH] chore: update comment --- packages/backend/src/apps/m365-excel/common/rate-limiter.ts | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/packages/backend/src/apps/m365-excel/common/rate-limiter.ts b/packages/backend/src/apps/m365-excel/common/rate-limiter.ts index c5fe8ffaa..be2b24a3b 100644 --- a/packages/backend/src/apps/m365-excel/common/rate-limiter.ts +++ b/packages/backend/src/apps/m365-excel/common/rate-limiter.ts @@ -99,8 +99,9 @@ export async function throttleStepsForPublishedPipes( throw new RetriableError({ error: 'Reached M365 step limit', - // IF we're rate limited, we're probably facing a spike of steps, so spread out - // retries over a wider time period to reduce the size of the thundering herd. + // If we're rate limited, we're probably facing a spike of steps for that + // file, so spread out retries over a wider time period (2x) to reduce the + // size of the retry thundering herd. delayInMs: P90_EXCEL_API_RTT_SECONDS * 1000 * 2, }) }