Skip to content

Commit

Permalink
test reverting WaitForCancellation: true
Browse files Browse the repository at this point in the history
  • Loading branch information
serprex committed Jan 6, 2024
1 parent a77938b commit ae10042
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 12 deletions.
14 changes: 5 additions & 9 deletions flow/workflows/cdc_flow.go
Original file line number Diff line number Diff line change
Expand Up @@ -216,8 +216,7 @@ func CDCFlowWorkflowWithConfig(
RetryPolicy: &temporal.RetryPolicy{
MaximumAttempts: 20,
},
SearchAttributes: mirrorNameSearch,
WaitForCancellation: true,
SearchAttributes: mirrorNameSearch,
}
setupFlowCtx := workflow.WithChildOptions(ctx, childSetupFlowOpts)
setupFlowFuture := workflow.ExecuteChildWorkflow(setupFlowCtx, SetupFlowWorkflow, cfg)
Expand All @@ -243,9 +242,8 @@ func CDCFlowWorkflowWithConfig(
RetryPolicy: &temporal.RetryPolicy{
MaximumAttempts: 20,
},
TaskQueue: taskQueue,
SearchAttributes: mirrorNameSearch,
WaitForCancellation: true,
TaskQueue: taskQueue,
SearchAttributes: mirrorNameSearch,
}
snapshotFlowCtx := workflow.WithChildOptions(ctx, childSnapshotFlowOpts)
snapshotFlowFuture := workflow.ExecuteChildWorkflow(snapshotFlowCtx, SnapshotFlowWorkflow, cfg)
Expand Down Expand Up @@ -394,8 +392,7 @@ func CDCFlowWorkflowWithConfig(
RetryPolicy: &temporal.RetryPolicy{
MaximumAttempts: 20,
},
SearchAttributes: mirrorNameSearch,
WaitForCancellation: true,
SearchAttributes: mirrorNameSearch,
}
syncCtx := workflow.WithChildOptions(ctx, childSyncFlowOpts)
syncFlowOptions.RelationMessageMapping = state.RelationMessageMapping
Expand Down Expand Up @@ -467,8 +464,7 @@ func CDCFlowWorkflowWithConfig(
RetryPolicy: &temporal.RetryPolicy{
MaximumAttempts: 20,
},
SearchAttributes: mirrorNameSearch,
WaitForCancellation: true,
SearchAttributes: mirrorNameSearch,
}
normCtx := workflow.WithChildOptions(ctx, childNormalizeFlowOpts)
childNormalizeFlowFuture := workflow.ExecuteChildWorkflow(
Expand Down
3 changes: 0 additions & 3 deletions flow/workflows/sync_flow.go
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,6 @@ func (s *SyncFlowExecution) executeSyncFlow(

syncMetaCtx := workflow.WithActivityOptions(ctx, workflow.ActivityOptions{
StartToCloseTimeout: 1 * time.Minute,
WaitForCancellation: true,
})

// execute GetLastSyncedID on destination peer
Expand All @@ -66,7 +65,6 @@ func (s *SyncFlowExecution) executeSyncFlow(
startFlowCtx := workflow.WithActivityOptions(ctx, workflow.ActivityOptions{
StartToCloseTimeout: 72 * time.Hour,
HeartbeatTimeout: 30 * time.Second,
WaitForCancellation: true,
})

// execute StartFlow on the peers to start the flow
Expand All @@ -85,7 +83,6 @@ func (s *SyncFlowExecution) executeSyncFlow(

replayTableSchemaDeltaCtx := workflow.WithActivityOptions(ctx, workflow.ActivityOptions{
StartToCloseTimeout: 30 * time.Minute,
WaitForCancellation: true,
})
replayTableSchemaInput := &protos.ReplayTableSchemaDeltaInput{
FlowConnectionConfigs: config,
Expand Down

0 comments on commit ae10042

Please sign in to comment.