Skip to content
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(zero-cache): wire pusher service through the existing ws connection #3778

Merged
merged 1 commit into from
Feb 12, 2025

Conversation

tantaman
Copy link
Contributor

The end state is to put the pusher/mutator on its own websocket connection (#3770).

That requires some refactoring of the client. Till that refactor is done (and so you all can use custom mutators while I'm on vacation), handle push messages over the existing websocket connection.

@tantaman tantaman requested a review from darkgnotic February 12, 2025 14:11
Copy link

vercel bot commented Feb 12, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
replicache-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 12, 2025 2:18pm
zbugs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 12, 2025 2:18pm

Copy link

github-actions bot commented Feb 12, 2025

🐰 Bencher Report

Branchmlaw/wire-pusher2
Testbedlocalhost
Click to view all benchmark results
BenchmarkFile SizeBenchmark Result
kilobytes (KB)
(Result Δ%)
Upper Boundary
kilobytes (KB)
(Limit %)
zero-package.tgz📈 view plot
🚷 view threshold
956.09
(+0.16%)Baseline: 954.54
973.63
(98.20%)
zero.js📈 view plot
🚷 view threshold
179.80
(0.00%)Baseline: 179.80
183.40
(98.04%)
zero.js.br📈 view plot
🚷 view threshold
50.01
(0.00%)Baseline: 50.01
51.01
(98.04%)
🐰 View full continuous benchmarking report in Bencher

Copy link

github-actions bot commented Feb 12, 2025

🐰 Bencher Report

Branchmlaw/wire-pusher2
Testbedlocalhost
Click to view all benchmark results
BenchmarkThroughputBenchmark Result
operations / second (ops/s)
(Result Δ%)
Lower Boundary
operations / second (ops/s)
(Limit %)
src/client/zero.bench.ts > basics > All 1000 rows x 10 columns (numbers)📈 view plot
🚷 view threshold
74.43
(-2.10%)Baseline: 76.03
70.38
(94.56%)
src/client/zero.bench.ts > with filter > Lower rows 500 x 10 columns (numbers)📈 view plot
🚷 view threshold
95.70
(+0.96%)Baseline: 94.79
90.44
(94.50%)
🐰 View full continuous benchmarking report in Bencher

…nection

The end state is to put the pusher/mutator on its own websocket connection (#3770).

This requires some refactoring of the client. Till that is done (and so you all can use custom mutators while I'm on vacation), handle push messages over the existing websocket connection.
@tantaman tantaman enabled auto-merge (rebase) February 12, 2025 14:20
@tantaman tantaman merged commit cd27a60 into main Feb 12, 2025
11 checks passed
@tantaman tantaman deleted the mlaw/wire-pusher2 branch February 12, 2025 14:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant