-
Notifications
You must be signed in to change notification settings - Fork 130
Graphile Queue Driver #467
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
base: main
Are you sure you want to change the base?
Conversation
🦋 Changeset detectedLatest commit: 3875887 The changes in this PR will be included in the next version bump. This PR includes changesets to release 16 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
@alex-holovach is attempting to deploy a commit to the Vercel Labs Team on Vercel. A member of the Team first needs to authorize it. |
9f9bab8 to
76031b2
Compare
packages/world-postgres/src/drizzle/migrations/0002_striped_lifeguard.sql
Outdated
Show resolved
Hide resolved
a48b48d to
6352633
Compare
de77fcd to
f5501c8
Compare
Signed-off-by: paulhenri-l <25308170+paulhenri-l@users.noreply.github.com> Signed-off-by: Alex Holovach <alex.holovach@kubiks.ai>
Signed-off-by: paulhenri-l <25308170+paulhenri-l@users.noreply.github.com> Signed-off-by: Alex Holovach <alex.holovach@kubiks.ai>
Signed-off-by: paulhenri-l <25308170+paulhenri-l@users.noreply.github.com> Signed-off-by: Alex Holovach <alex.holovach@kubiks.ai>
Signed-off-by: paulhenri-l <25308170+paulhenri-l@users.noreply.github.com> Signed-off-by: Alex Holovach <alex.holovach@kubiks.ai>
Signed-off-by: paulhenri-l <25308170+paulhenri-l@users.noreply.github.com> Signed-off-by: Alex Holovach <alex.holovach@kubiks.ai>
Signed-off-by: paulhenri-l <25308170+paulhenri-l@users.noreply.github.com> Signed-off-by: Alex Holovach <alex.holovach@kubiks.ai>
Signed-off-by: paulhenri-l <25308170+paulhenri-l@users.noreply.github.com> Signed-off-by: Alex Holovach <alex.holovach@kubiks.ai>
Signed-off-by: paulhenri-l <25308170+paulhenri-l@users.noreply.github.com> Signed-off-by: Alex Holovach <alex.holovach@kubiks.ai>
Signed-off-by: paulhenri-l <25308170+paulhenri-l@users.noreply.github.com> Signed-off-by: Alex Holovach <alex.holovach@kubiks.ai>
Signed-off-by: paulhenri-l <25308170+paulhenri-l@users.noreply.github.com> Signed-off-by: Alex Holovach <alex.holovach@kubiks.ai>
Signed-off-by: paulhenri-l <25308170+paulhenri-l@users.noreply.github.com> Signed-off-by: Alex Holovach <alex.holovach@kubiks.ai>
Signed-off-by: paulhenri-l <25308170+paulhenri-l@users.noreply.github.com> Signed-off-by: Alex Holovach <alex.holovach@kubiks.ai>
Signed-off-by: Alex Holovach <alex.holovach@kubiks.ai>
Signed-off-by: Alex Holovach <alex.holovach@kubiks.ai>
Signed-off-by: Alex Holovach <alex.holovach@kubiks.ai>
Signed-off-by: Alex Holovach <alex.holovach@kubiks.ai>
Signed-off-by: Alex Holovach <alex.holovach@kubiks.ai>
Restores the initial attempt value to 1 to match main branch behavior and the schema contract (StepRetryingEventSchema requires attempt >= 1). Signed-off-by: Alex Holovach <alex.holovach@kubiks.ai>
* fix(utils): detect linux ports via /proc Signed-off-by: Alex Holovach <alex.holovach@kubiks.ai> * extract parsePort helper Signed-off-by: Alex Holovach <alex.holovach@kubiks.ai> * add changeset Signed-off-by: Alex Holovach <alex.holovach@kubiks.ai> --------- Signed-off-by: Alex Holovach <alex.holovach@kubiks.ai>
…rt support Signed-off-by: Alex Holovach <alex.holovach@kubiks.ai>
Signed-off-by: Alex Holovach <alex.holovach@kubiks.ai>
f5501c8 to
3875887
Compare
|
Create PR Comment job fails with error:
@pranaygp Is it missing some permissions? |
| // Backend can be: local, postgres, postgres-pgboss, postgres-graphile, vercel | ||
| const knownBackends = [ |
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.
I'm actively making improvements to the benchmarking stuff so expect some merge conflicts on adding your own stuff to benchmark
|
I don't think we're going to "queue driver" approach (see #332 (comment)) - it defeats the point of the world abstraction if graphile is a good replacement for pgboss and is faster, then I'm down to just swap out the pg boss implementation in postgres world to use graphile. I added new benchmarks too, so if you redo the PR against the latest main and just swap out the implementation to use graphile without changing the benchmark code, I can trigger the CI testing and benchmarking for you |
|
Agreed, exposing the queue driver adds complexity without clear value for users in this case. I was exploring this to see if we could get lower latency for the Postgres world. Let's see how #332 lands, and I'll revisit this based on that direction. |

Added Graphile Worker as an alternative queue driver for
@workflow/world-postgres, building on top of the queue driver abstraction introduced in #332 .Usage
Automatic selection via env var
Or explicit factory