Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
PR SummaryMedium Risk Overview Reviewed by Cursor Bugbot for commit 94e0e62. Bugbot is set up for automated code reviews on this repo. Configure here. |
Greptile SummaryThis PR increases the postgres.js connection pool Confidence Score: 5/5Safe to merge — config-only change with no logic impact and a well-understood rationale (PgBouncer transaction mode). Both changes are single-value config bumps with no code logic affected. The asymmetry (20 vs 25) appears intentional given the socket server's higher per-instance concurrency requirements. No P0/P1 findings. No files require special attention. Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[App Instances] -->|"max: 20 connections each"| B[PgBouncer\nTransaction Mode]
C[Socket Server] -->|"max: 25 connections each"| B
B -->|"Multiplexed server connections"| D[(PostgreSQL)]
style B fill:#f0a500,color:#000
style D fill:#336791,color:#fff
Reviews (1): Last reviewed commit: "Raise socket connections" | Re-trigger Greptile |
Summary
Previously we decreased our max db connections per task. This caused issues scaling when we were hit with a sudden spike in network requests. With transaction mode pgbouncer we can afford to have higher numbers here.
Fixes #(issue)
Type of Change
Testing
Checklist
Screenshots/Videos