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(orchestration): introducing the scheduler #2132
Conversation
59b50d5
to
ff4b239
Compare
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.
Massive 👏🏻
Did not had time to test everything, but here is my first batch of comments
packages/scheduler/lib/db/migrations/20240506105059_initial_scheduler_models.ts
Outdated
Show resolved
Hide resolved
packages/scheduler/lib/db/migrations/20240506105059_initial_scheduler_models.ts
Outdated
Show resolved
Hide resolved
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.
Finished reading, I'll let you reply and adjust what you feel relevant. Ping me when you want a second review
ff4b239
to
ad1e099
Compare
I addressed all your comments @bodinsamuel |
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.
Let's go 🚀
ad1e099
to
1fe2f9d
Compare
}) | ||
.forUpdate() | ||
.skipLocked() | ||
.debug(true) |
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.
Intentional I presume to leave this in while under development @TBonnin ?
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.
no actually it is some dev leftover. I will remove it
Looks good! Tests 💯 |
This is the first stage of the orchestration v2, implementing the scheduler.
As agreed the first version of the scheduler doesn't support recurring tasks, only tasks that are scheduled immediately.
scheduler
package is not used yet but it is big enough to warrant a review. There is still a few TODOs, which I will address in subsequent PRsIssue ticket number and link
https://linear.app/nango/document/[brief]-orchestration-v2-4c5d1f1e104b#94b7a156-de22-4ce2-8a77-9cfcfe1fcece
Checklist before requesting a review (skip if just adding/editing APIs & templates)