Skip to content

Commit 922c7cb

Browse files
authored
Increase batch trigger processing job visibility timeout (#2234)
1 parent 0fcb970 commit 922c7cb

File tree

2 files changed

+7
-2
lines changed

2 files changed

+7
-2
lines changed

apps/webapp/app/env.server.ts

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -717,6 +717,11 @@ const EnvironmentSchema = z.object({
717717
COMMON_WORKER_REDIS_TLS_DISABLED: z.string().default(process.env.REDIS_TLS_DISABLED ?? "false"),
718718
COMMON_WORKER_REDIS_CLUSTER_MODE_ENABLED: z.string().default("0"),
719719

720+
BATCH_TRIGGER_PROCESS_JOB_VISIBILITY_TIMEOUT_MS: z.coerce
721+
.number()
722+
.int()
723+
.default(60_000 * 5), // 5 minutes
724+
720725
BATCH_TRIGGER_WORKER_ENABLED: z.string().default(process.env.WORKER_ENABLED ?? "true"),
721726
BATCH_TRIGGER_WORKER_CONCURRENCY_WORKERS: z.coerce.number().int().default(2),
722727
BATCH_TRIGGER_WORKER_CONCURRENCY_TASKS_PER_WORKER: z.coerce.number().int().default(10),

apps/webapp/app/v3/batchTriggerWorker.server.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ function initializeWorker() {
3434
attemptCount: z.number().int(),
3535
strategy: z.enum(["sequential", "parallel"]),
3636
}),
37-
visibilityTimeoutMs: 60_000,
37+
visibilityTimeoutMs: env.BATCH_TRIGGER_PROCESS_JOB_VISIBILITY_TIMEOUT_MS,
3838
retry: {
3939
maxAttempts: 5,
4040
},
@@ -49,7 +49,7 @@ function initializeWorker() {
4949
parentRunId: z.string().optional(),
5050
resumeParentOnCompletion: z.boolean().optional(),
5151
}),
52-
visibilityTimeoutMs: 60_000,
52+
visibilityTimeoutMs: env.BATCH_TRIGGER_PROCESS_JOB_VISIBILITY_TIMEOUT_MS,
5353
retry: {
5454
maxAttempts: 5,
5555
},

0 commit comments

Comments
 (0)