Skip to content

Commit df70bac

Browse files
authored
Added request ID transition option when creating activity entity. (#8647)
## What changed? Added request ID transition option when creating activity entity. ## Why? The requestID is set by the matching service to a UUID, allowing safe retries if the response is lost. Here we add the request ID transition option so chasm engine handles it for us under the hood. ## How did you test it? - [X] built - [X] run locally and tested manually - [X] covered by existing tests - [ ] added new unit test(s) - [ ] added new functional test(s)
1 parent 2067756 commit df70bac

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

chasm/lib/activity/handler.go

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,9 @@ func (h *handler) StartActivityExecution(ctx context.Context, req *activitypb.St
3939
// EagerTask: TODO when supported, need to call the same code that would handle the RecordActivityTaskStarted API
4040
}, nil
4141
},
42-
req.GetFrontendRequest())
42+
req.GetFrontendRequest(),
43+
chasm.WithRequestID(req.GetFrontendRequest().GetRequestId()),
44+
)
4345

4446
if err != nil {
4547
return nil, err

0 commit comments

Comments
 (0)