-
Notifications
You must be signed in to change notification settings - Fork 18
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Notification system #65
Comments
This is much more complicated than it sounds and we should discuss this at a broader level. gRPC streaming is a bit hard to proxy properly in a truly HA environment. Also, lossless stream resumption across workflows would be fraught with state complication. Also, maintaining sinks to user endpoints that you push to from anywhere in the cluster when an event happens is rough from a state management POV. Also, there is no need for the UI or condition builder or flexible action or any of those other features for some kind of MVP. Many users have a larger use case than what you are asking here: They want every Temporal event from every workflow or activity or even cloud events (but some of their use cases can be solved by export). It'd make little sense to have some single-workflow callback and ignore the ability to provide all Temporal events. We need a general firehose that anyone read from, but again there are many complications involved. It should be a full project and not narrowly tailored. I have many more advanced implementation ideas for this and would be willing to submit a real proposal if requested. There has also been some discussion this in the past, so those points would need to be included.
That's like the entire part :-) |
👍 I meant the first as the firehose (with which someone could implement the second externally). Makes sense to include activity & cloud events as well. Product-wise, I think an MVP could either be the firehose or a small set of conditions with a webhook action. |
A firehose of events that spans all Temporal workflows/activities would be very useful to us. If it's helpful motivation for writing the proposal, here are two use-cases that come to mind:
|
Another use case: https://docs.prefect.io/ui/automations/ |
Could also add Cloud-specific events like namespace creation:
|
Author: Loren
Summary of the feature being proposed
Add two capabilities:
What value does this feature bring to Temporal?
Users want to be able to receive a push notification when something happens. For example, when a Workflow times out, they want to receive that event with information about the Workflow, like its ID and history.
The current methods of getting this are:
temporal_cloud_v0_workflow_timeout_count
ListWorkflowExecutions
for Executions in a Timed Out state (note: polling may take more steps and be more resource-intensive to do for other types of events)Are you willing to implement this feature yourself?
Yes but I'd be pretty slow at the Go part 😄
The text was updated successfully, but these errors were encountered: