Skip to content

Commit 53d2ffc

Browse files
committed
chore(run-engine): improve concurrency sweeper logging to get better visibility
1 parent 8313800 commit 53d2ffc

File tree

1 file changed

+12
-8
lines changed
  • internal-packages/run-engine/src/run-queue

1 file changed

+12
-8
lines changed

internal-packages/run-engine/src/run-queue/index.ts

Lines changed: 12 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1862,10 +1862,12 @@ export class RunQueue {
18621862
// Call this every 10 minutes
18631863
private async scanConcurrencySets() {
18641864
if (this.abortController.signal.aborted) {
1865+
this.logger.info("Abort signal received, skipping concurrency scan");
1866+
18651867
return;
18661868
}
18671869

1868-
this.logger.debug("Scanning concurrency sets for completed runs");
1870+
this.logger.info("Scanning concurrency sets for completed runs");
18691871

18701872
const stats = {
18711873
streamCallbacks: 0,
@@ -1918,7 +1920,7 @@ export class RunQueue {
19181920
return;
19191921
}
19201922

1921-
this.logger.debug("Processing concurrency keys from stream", {
1923+
this.logger.info("Processing concurrency keys from stream", {
19221924
keys: uniqueKeys,
19231925
});
19241926

@@ -1988,27 +1990,29 @@ export class RunQueue {
19881990
}
19891991

19901992
private async processCurrentConcurrencyRunIds(concurrencyKey: string, runIds: string[]) {
1991-
this.logger.debug(`Processing concurrency set with ${runIds.length} runs`, {
1993+
this.logger.info("Processing concurrency set with runs", {
19921994
concurrencyKey,
1993-
runIds: runIds.slice(0, 5), // Log first 5 for debugging
1995+
runIds: runIds.slice(0, 5), // Log first 5 for debugging,
1996+
runIdsLength: runIds.length,
19941997
});
19951998

19961999
// Call the callback to determine which runs are completed
19972000
const completedRuns = await this.options.concurrencySweeper?.callback(runIds);
19982001

19992002
if (!completedRuns) {
2000-
this.logger.debug("No completed runs found in concurrency set", { concurrencyKey });
2003+
this.logger.info("No completed runs found in concurrency set", { concurrencyKey });
20012004
return;
20022005
}
20032006

20042007
if (completedRuns.length === 0) {
2005-
this.logger.debug("No completed runs found in concurrency set", { concurrencyKey });
2008+
this.logger.info("No completed runs found in concurrency set", { concurrencyKey });
20062009
return;
20072010
}
20082011

2009-
this.logger.debug(`Found ${completedRuns.length} completed runs to mark for ack`, {
2012+
this.logger.info("Found completed runs to mark for ack", {
20102013
concurrencyKey,
20112014
completedRunIds: completedRuns.map((r) => r.id).slice(0, 5),
2015+
completedRunIdsLength: completedRuns.length,
20122016
});
20132017

20142018
// Mark the completed runs for acknowledgment
@@ -2032,7 +2036,7 @@ export class RunQueue {
20322036

20332037
const count = await this.redis.zadd(markedForAckKey, ...args);
20342038

2035-
this.logger.debug(`Marked ${count} runs for acknowledgment`, {
2039+
this.logger.info("Marked runs for acknowledgment", {
20362040
markedForAckKey,
20372041
count,
20382042
});

0 commit comments

Comments
 (0)