uber-go / cadence-client

Framework for authoring workflows and activities running on top of the Cadence orchestration engine.
https://cadenceworkflow.io
MIT License
345 stars 131 forks source link

Add DecisionTaskStartedEventID to workflowInfo #1127

Closed longquanzheng closed 3 years ago

longquanzheng commented 3 years ago

What changed? Add DecisionTaskStartedEventID to workflowInfo

Why? In some use cases, users need to get the eventID for resetting certain activity/timer/etc. This is the similar to BinaryChecksum which we are currently supporting.

How did you test it? Existing tests

Potential risks No

coveralls commented 3 years ago

Pull Request Test Coverage Report for Build 27f52762-e037-41ba-80a8-a3d049506fae


Changes Missing Coverage Covered Lines Changed/Added Lines %
internal/workflow.go 0 3 0.0%
<!-- Total: 2 5 40.0% -->
Files with Coverage Reduction New Missed Lines %
internal/internal_task_pollers.go 2 81.53%
internal/compatibility/response.go 11 48.43%
<!-- Total: 13 -->
Totals Coverage Status
Change from base Build 2e827765-7cf1-4470-a352-de8ddaf1cfe7: 0.1%
Covered Lines: 12188
Relevant Lines: 17010

💛 - Coveralls