-
Notifications
You must be signed in to change notification settings - Fork 191
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
feat(v1): new gRPC API endpoints #1367
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
||
message SleepMatchCondition { | ||
BaseMatchCondition base = 1; | ||
string sleep_for = 2; // a duration string indicating how long to sleep |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
just curious what resolution can we support (i.e. ms?)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Go duration string: Valid time units are "ns", "us" (or "µs"), "ms", "s", "m", "h"
message CreateTaskOpts { | ||
string readable_id = 1; // (required) the task name | ||
string action = 2; // (required) the task action id | ||
string timeout = 3; // (optional) the task timeout |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is schedule time out not on the task now?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice catch, it is
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe schedule_timeout on the workflow should be changed to default_schedule timeout
also priority is task level
} | ||
|
||
message BaseMatchCondition { | ||
string event_key = 1; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removing
string readable_data_key = 2; | ||
Action action = 3; | ||
string or_group_id = 4; // a UUID defining the OR group for this condition | ||
string expression = 5; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
optional
|
||
enum Action { | ||
CREATE = 0; | ||
QUEUE = 1; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Document: QUEUE, CANCEL, SKIP relevant for task triggers, CREATE relevant for signals
Description
Adds endpoints for durable event listeners and more dynamic DAG triggering/skipping.
Type of change