From 83a8adf0332a09561121cd86c74983fd15d308f9 Mon Sep 17 00:00:00 2001 From: Audric Ackermann Date: Wed, 5 Jul 2023 09:14:04 +0200 Subject: [PATCH] fix: decrease retry timer of confsyncjob to 15s - we need a rather long duration so we don't add a new job while was is already planned to be runned, and they end up both running at the same time. - we also need a not too long duration for integration tests and UX experience, as a change could take the retry timeout to happen+polling timer to happen, which used to be 40s total. This brings that time to worse time change from 40 to 25s --- ts/session/utils/job_runners/jobs/ConfigurationSyncJob.ts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ts/session/utils/job_runners/jobs/ConfigurationSyncJob.ts b/ts/session/utils/job_runners/jobs/ConfigurationSyncJob.ts index 0cef20c86..5fc60cbf9 100644 --- a/ts/session/utils/job_runners/jobs/ConfigurationSyncJob.ts +++ b/ts/session/utils/job_runners/jobs/ConfigurationSyncJob.ts @@ -20,7 +20,7 @@ import { ReleasedFeatures } from '../../../../util/releaseFeature'; import { allowOnlyOneAtATime } from '../../Promise'; import { isSignInByLinking } from '../../../../util/storage'; -const defaultMsBetweenRetries = 30000; // a long time between retries, to avoid running multiple jobs at the same time, when one was postponed at the same time as one already planned (5s) +const defaultMsBetweenRetries = 15000; // a long time between retries, to avoid running multiple jobs at the same time, when one was postponed at the same time as one already planned (5s) const defaultMaxAttempts = 2; /**