I would like to have a way to get the state (percentage of completeness or other) from an activity to the workflow. So the workflow can return it as the part of a query to the workflow or use it for reasoning on the rest of the execution. Now I see two ways:
Activity heartbeats can have some data (percentage or any other). But I cannot find a way to read about the activity heartbeats from the workflow.
Signals. An activity can signal some data to the workflow as some milestones reached by the activity (every 10 percent, for example). And the workflow will be able to read the data and reason or send it as the result of the query.
From outside of workflow you can use DescribeWorkflowExecution call that returns all pending activities information including data included in the last heartbeat.
I would go with the signal approach you described if the workflow has to take action based on activity reaching a certain milestone.
It is not clear what is the best way to receive the signals from activities in a correct way. Now I use workflow.Await() inside the workflow.Go()temporal-status/workflow.go at main · guntenbein/temporal-status · GitHub and it works, the coroutine finishes so there are no memory leaks. But there is the comment for the Await : Do not mutate values or trigger side effects inside condition.
The ReceiveAsync of the channel cannot be used because of the Coroutine panics as blocked. It happens because ReceiveAsync does invoke yeld() for the coroutine state.
Agree that the Await approach is not correct as it mutates the workflow state from the condition function.
The second approach with blocking on the Channel.Receive looks fine to me. I’m not sure why you treat the goroutine as a memory leak. When the workflow completes it will be cleaned up as well.
Another more common approach is to use Selector that listens on multiple channels and Futures (which can be results of activity and child workflow invocations).