Skip to content

Commit c16430d

Browse files
authored
Change 30 min retry policies to 20 min (#3597)
We've changed the recoverable monitor to anticipate ~20 minute max gaps between errors. Accommodating in the code before the upgrades.
1 parent 487b5b6 commit c16430d

File tree

2 files changed

+8
-8
lines changed

2 files changed

+8
-8
lines changed

flow/workflows/qrep_flow.go

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -78,7 +78,7 @@ func (q *QRepFlowExecution) SetupMetadataTables(ctx workflow.Context) error {
7878
RetryPolicy: &temporal.RetryPolicy{
7979
InitialInterval: time.Minute,
8080
BackoffCoefficient: 2.,
81-
MaximumInterval: 30 * time.Minute,
81+
MaximumInterval: 20 * time.Minute,
8282
MaximumAttempts: 0,
8383
NonRetryableErrorTypes: nil,
8484
},
@@ -100,7 +100,7 @@ func (q *QRepFlowExecution) setupTableSchema(ctx workflow.Context, tableName str
100100
RetryPolicy: &temporal.RetryPolicy{
101101
InitialInterval: time.Minute,
102102
BackoffCoefficient: 2.,
103-
MaximumInterval: 30 * time.Minute,
103+
MaximumInterval: 20 * time.Minute,
104104
MaximumAttempts: 0,
105105
NonRetryableErrorTypes: nil,
106106
},
@@ -132,7 +132,7 @@ func (q *QRepFlowExecution) setupWatermarkTableOnDestination(ctx workflow.Contex
132132
RetryPolicy: &temporal.RetryPolicy{
133133
InitialInterval: time.Minute,
134134
BackoffCoefficient: 2.,
135-
MaximumInterval: 30 * time.Minute,
135+
MaximumInterval: 20 * time.Minute,
136136
MaximumAttempts: 0,
137137
NonRetryableErrorTypes: nil,
138138
},
@@ -293,7 +293,7 @@ func (q *QRepFlowExecution) consolidatePartitions(ctx workflow.Context) error {
293293
RetryPolicy: &temporal.RetryPolicy{
294294
InitialInterval: time.Minute,
295295
BackoffCoefficient: 2.,
296-
MaximumInterval: 30 * time.Minute,
296+
MaximumInterval: 20 * time.Minute,
297297
MaximumAttempts: 0,
298298
NonRetryableErrorTypes: nil,
299299
},
@@ -358,7 +358,7 @@ func (q *QRepFlowExecution) handleTableCreationForResync(ctx workflow.Context, s
358358
RetryPolicy: &temporal.RetryPolicy{
359359
InitialInterval: time.Minute,
360360
BackoffCoefficient: 2.,
361-
MaximumInterval: 30 * time.Minute,
361+
MaximumInterval: 20 * time.Minute,
362362
MaximumAttempts: 0,
363363
NonRetryableErrorTypes: nil,
364364
},
@@ -404,7 +404,7 @@ func (q *QRepFlowExecution) handleTableRenameForResync(ctx workflow.Context, sta
404404
RetryPolicy: &temporal.RetryPolicy{
405405
InitialInterval: time.Minute,
406406
BackoffCoefficient: 2.,
407-
MaximumInterval: 30 * time.Minute,
407+
MaximumInterval: 20 * time.Minute,
408408
MaximumAttempts: 0,
409409
NonRetryableErrorTypes: nil,
410410
},
@@ -445,7 +445,7 @@ func QRepWaitForNewRowsWorkflow(ctx workflow.Context, config *protos.QRepConfig,
445445
RetryPolicy: &temporal.RetryPolicy{
446446
InitialInterval: time.Minute,
447447
BackoffCoefficient: 2.,
448-
MaximumInterval: 30 * time.Minute,
448+
MaximumInterval: 20 * time.Minute,
449449
MaximumAttempts: 0,
450450
NonRetryableErrorTypes: nil,
451451
},

flow/workflows/xmin_flow.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -77,7 +77,7 @@ func XminFlowWorkflow(
7777
RetryPolicy: &temporal.RetryPolicy{
7878
InitialInterval: time.Minute,
7979
BackoffCoefficient: 2.,
80-
MaximumInterval: 30 * time.Minute,
80+
MaximumInterval: 20 * time.Minute,
8181
MaximumAttempts: 0,
8282
NonRetryableErrorTypes: nil,
8383
},

0 commit comments

Comments
 (0)