chore(pegboard): consolidate to single subscriber per gateway #2879

NathanFlurry · 2025-09-08T05:13:14Z

Fixes RVT-5133

Fixes RVT-5120

vercel · 2025-09-08T05:13:23Z

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Preview	Comments	Updated (UTC)
rivet-cloud	Error			Sep 16, 2025 8:34am
rivet-site	Ready	Preview	Comment	Sep 16, 2025 8:34am
rivet-studio	Ready	Preview	Comment	Sep 16, 2025 8:34am

NathanFlurry · 2025-09-08T05:13:27Z

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

How to use the Graphite Merge Queue

Add the label merge-queue to this PR to add it to the merge queue.

You must have a Graphite account in order to use the merge queue. Sign up using this link.

_{An organization admin has enabled the Graphite Merge Queue in this repository.} _{Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.}

This stack of pull requests is managed by Graphite. Learn more about stacking.

linear · 2025-09-08T05:22:13Z

RVT-5133 Update & document tunnel arch

claude · 2025-09-08T05:30:49Z

Claude encountered an error —— View job

I'll analyze this and get back to you.

claude · 2025-09-08T05:45:17Z

Claude encountered an error —— View job

I'll analyze this and get back to you.

claude · 2025-09-08T07:04:21Z

Claude encountered an error —— View job

I'll analyze this and get back to you.

linear · 2025-09-08T07:06:11Z

RVT-5120 Move UPS response error handler to a diff channel

MasterPtato

I dont understand why we need a single sub for the gateway

MasterPtato · 2025-09-08T21:54:43Z

packages/common/universalpubsub/src/driver/postgres/mod.rs

@@ -40,7 +49,7 @@ pub const POSTGRES_MAX_MESSAGE_SIZE: usize =
 pub struct PostgresDriver {
 	pool: Arc<Pool>,
 	client: Arc<tokio_postgres::Client>,
-	subscriptions: Cache<String, Subscription>,
+	subscriptions: Arc<Mutex<HashMap<String, Subscription>>>,


I dont see why this change was made, it makes contention a much higher probability

Misunderstood what Cache was. Will fix in RVT-5138

NathanFlurry · 2025-09-08T23:30:06Z

I dont understand why we need a single sub for the gateway

Reduced overhead for starting requests bc you dont need to create an extra sub. I think NATS is optimized so this isn't as big a deal but PG it certainly is

claude · 2025-09-09T03:30:43Z

Claude encountered an error —— View job

I'll analyze this and get back to you.

claude · 2025-09-09T03:30:45Z

Claude encountered an error —— View job

I'll analyze this and get back to you.

claude · 2025-09-09T03:38:19Z

Claude encountered an error —— View job

I'll analyze this and get back to you.

claude · 2025-09-14T20:03:54Z

Claude encountered an error —— View job

I'll analyze this and get back to you.

claude · 2025-09-15T16:49:44Z

Claude encountered an error —— View job

I'll analyze this and get back to you.

claude · 2025-09-15T18:06:18Z

Claude encountered an error —— View job

I'll analyze this and get back to you.

…ubscriber_per_gateway

claude · 2025-09-16T08:01:53Z

Claude encountered an error —— View job

I'll analyze this and get back to you.

NathanFlurry requested a review from MasterPtato September 8, 2025 05:13

This was referenced Sep 8, 2025

chore(runner): connect tunnel before connecting pb ws #2872

Closed

chore(ups): add benches #2878

Closed

This was referenced Sep 8, 2025

fix(pegboard): fix actor names not reserving on first reservation #2861

Closed

chore(ups): add message chunking & ups protocol #2874

Closed

chore(runner): disable default logs #2862

Closed

chore(ups): remove no responders #2863

Closed

vercel bot had a problem deploying to Preview – rivet-site September 8, 2025 05:30 Failure

NathanFlurry force-pushed the 09-06-chore_pegboard_consolidate_to_single_subscriber_per_gateway branch from 4881e7e to 34cf689 Compare September 8, 2025 07:04

vercel bot had a problem deploying to Preview – rivet-site September 8, 2025 07:04 Failure

MasterPtato requested changes Sep 8, 2025

View reviewed changes

NathanFlurry force-pushed the 09-06-chore_pegboard_consolidate_to_single_subscriber_per_gateway branch from 34cf689 to 858c193 Compare September 9, 2025 03:30

NathanFlurry force-pushed the 09-06-chore_ups_add_benches branch from 3586f1c to f68b17f Compare September 9, 2025 03:30

This was referenced Sep 9, 2025

chore(ups): add postgres auto-reconnect #2892

Merged

chore(ups): handle edge cases with postgres listen/unlisten/notify when disconnected/reconnecting #2893

Closed

NathanFlurry requested a review from MasterPtato September 9, 2025 03:34

vercel bot had a problem deploying to Preview – rivet-site September 9, 2025 03:36 Failure

NathanFlurry force-pushed the 09-06-chore_pegboard_consolidate_to_single_subscriber_per_gateway branch from 858c193 to c84e8b5 Compare September 9, 2025 03:38

NathanFlurry force-pushed the 09-06-chore_ups_add_benches branch from f68b17f to d6b0b65 Compare September 9, 2025 03:38

NathanFlurry force-pushed the 09-06-chore_pegboard_consolidate_to_single_subscriber_per_gateway branch from 0ecbe78 to 5e0abe2 Compare September 14, 2025 20:00

vercel bot had a problem deploying to Preview – rivet-cloud September 14, 2025 20:32 Failure

vercel bot deployed to Preview – rivet-site September 14, 2025 20:35 View deployment

NathanFlurry mentioned this pull request Sep 14, 2025

chore(runner): add alarms api #2919

Merged

jog1t force-pushed the 09-06-chore_pegboard_consolidate_to_single_subscriber_per_gateway branch from 5e0abe2 to 0ecbe78 Compare September 15, 2025 16:47

jog1t mentioned this pull request Sep 15, 2025

feat(inspector): make inspector compatible with new architecture #2920

Closed

chore(pegboard): consolidate to single subscriber per gateway

9d88491

MasterPtato removed their request for review September 15, 2025 17:55

MasterPtato force-pushed the 09-06-chore_pegboard_consolidate_to_single_subscriber_per_gateway branch from 0ecbe78 to 9d88491 Compare September 15, 2025 18:04

MasterPtato mentioned this pull request Sep 15, 2025

chore: fix api gen #2921

Merged

vercel bot deployed to Preview – rivet-site September 15, 2025 18:26 View deployment

vercel bot had a problem deploying to Preview – rivet-cloud September 15, 2025 18:27 Failure

MasterPtato mentioned this pull request Sep 15, 2025

chore: clean up api-public #2922

Merged

Merge branch 'main' into 09-06-chore_pegboard_consolidate_to_single_s…

b3ed2a4

…ubscriber_per_gateway

NathanFlurry merged commit 2916549 into main Sep 16, 2025
3 of 10 checks passed

NathanFlurry deleted the 09-06-chore_pegboard_consolidate_to_single_subscriber_per_gateway branch September 16, 2025 08:01

vercel bot had a problem deploying to Preview – rivet-cloud September 16, 2025 08:02 Failure

github-actions bot mentioned this pull request Sep 16, 2025

chore(main): release 25.6.1 #2851

Merged

vercel bot deployed to Preview – rivet-site September 16, 2025 08:32 View deployment

vercel bot deployed to Preview – rivet-studio September 16, 2025 08:34 View deployment

chore(pegboard): consolidate to single subscriber per gateway #2879

chore(pegboard): consolidate to single subscriber per gateway #2879

Uh oh!

Conversation

NathanFlurry commented Sep 8, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

vercel bot commented Sep 8, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

NathanFlurry commented Sep 8, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

How to use the Graphite Merge Queue

Uh oh!

linear bot commented Sep 8, 2025

Uh oh!

claude bot commented Sep 8, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

claude bot commented Sep 8, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

claude bot commented Sep 8, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

linear bot commented Sep 8, 2025

Uh oh!

MasterPtato left a comment

Choose a reason for hiding this comment

Uh oh!

MasterPtato Sep 8, 2025

Choose a reason for hiding this comment

Uh oh!

NathanFlurry Sep 9, 2025

Choose a reason for hiding this comment

Uh oh!

NathanFlurry commented Sep 8, 2025

Uh oh!

claude bot commented Sep 9, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

claude bot commented Sep 9, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

claude bot commented Sep 9, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

claude bot commented Sep 14, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

claude bot commented Sep 15, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

claude bot commented Sep 15, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

claude bot commented Sep 16, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

NathanFlurry commented Sep 8, 2025 •

edited

Loading

vercel bot commented Sep 8, 2025 •

edited

Loading

NathanFlurry commented Sep 8, 2025 •

edited

Loading

claude bot commented Sep 8, 2025 •

edited

Loading

claude bot commented Sep 8, 2025 •

edited

Loading

claude bot commented Sep 8, 2025 •

edited

Loading

claude bot commented Sep 9, 2025 •

edited

Loading

claude bot commented Sep 9, 2025 •

edited

Loading

claude bot commented Sep 9, 2025 •

edited

Loading

claude bot commented Sep 14, 2025 •

edited

Loading

claude bot commented Sep 15, 2025 •

edited

Loading

claude bot commented Sep 15, 2025 •

edited

Loading

claude bot commented Sep 16, 2025 •

edited

Loading