Skip to content

Commit 26b63d6

Browse files
authored
fix(session-ingest): extend queue retry delay (#3709)
1 parent 34fce77 commit 26b63d6

2 files changed

Lines changed: 10 additions & 3 deletions

File tree

services/session-ingest/src/queue-consumer.test.ts

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,12 @@ vi.mock('./util/ingest-limits', () => ({
4444
import { getWorkerDb } from '@kilocode/db/client';
4545
import { getSessionIngestDO } from './dos/SessionIngestDO';
4646
import { notifyUserSessionEvent } from './session-events';
47-
import { computeSessionMetadataUpdates, createItemExtractor, queue } from './queue-consumer';
47+
import {
48+
QUEUE_RETRY_DELAY_SECONDS,
49+
computeSessionMetadataUpdates,
50+
createItemExtractor,
51+
queue,
52+
} from './queue-consumer';
4853

4954
const encoder = new TextEncoder();
5055

@@ -176,7 +181,7 @@ describe('queue', () => {
176181
);
177182

178183
expect(ack).not.toHaveBeenCalled();
179-
expect(retry).toHaveBeenCalledWith({ delaySeconds: 60 });
184+
expect(retry).toHaveBeenCalledWith({ delaySeconds: QUEUE_RETRY_DELAY_SECONDS });
180185
});
181186

182187
it('passes full parsed oversized message data and its R2 reference into ingest', async () => {

services/session-ingest/src/queue-consumer.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,8 @@ export interface IngestQueueMessage {
2020
ingestedAt: number;
2121
}
2222

23+
export const QUEUE_RETRY_DELAY_SECONDS = 5 * 60;
24+
2325
function elapsedMs(startedAt: number): number {
2426
return Date.now() - startedAt;
2527
}
@@ -540,7 +542,7 @@ export async function queue(
540542
sessionId: msg.body.sessionId,
541543
error: err instanceof Error ? err.message : String(err),
542544
});
543-
msg.retry({ delaySeconds: 60 });
545+
msg.retry({ delaySeconds: QUEUE_RETRY_DELAY_SECONDS });
544546
}
545547
}
546548
}

0 commit comments

Comments
 (0)