Skip to content

Revert: Reduce DB pool max from 100 to 15#555

Closed
iscekic wants to merge 1 commit intomainfrom
revert/reduce-db-pool-max
Closed

Revert: Reduce DB pool max from 100 to 15#555
iscekic wants to merge 1 commit intomainfrom
revert/reduce-db-pool-max

Conversation

@iscekic
Copy link
Contributor

@iscekic iscekic commented Feb 25, 2026

Summary

When to use

Merge this if the pool max reduction in #554 causes connection timeout errors due to pools being too small under sustained high load.

@iscekic iscekic self-assigned this Feb 25, 2026
@kilo-code-bot
Copy link
Contributor

kilo-code-bot bot commented Feb 25, 2026

Code Review Summary

Status: No Issues Found | Recommendation: Merge (when needed)

This is a pre-prepared revert PR that changes the DB pool max from 15 back to 100 for both primary and replica pools, undoing #554. The PR description correctly notes it should only be merged if #554 causes connection timeout issues.

The changes are minimal and correct:

  • max: 15max: 100 on both pools
  • Removed the explanatory comment about why 15 was chosen (appropriate since the value is being reverted)

No bugs, security issues, or logic errors found.

Files Reviewed (1 file)
  • src/lib/drizzle.ts - pool max reverted from 15 to 100 on both primary and replica pools

@iscekic iscekic requested a review from RSO February 25, 2026 12:04
Copy link
Contributor

@RSO RSO left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No changes to show

LGTM ;P

@iscekic
Copy link
Contributor Author

iscekic commented Feb 25, 2026

Best PR 😂

…ad (#554)"

This reverts commit af410e9, reversing
changes made to 64d8be5.
@iscekic iscekic force-pushed the revert/reduce-db-pool-max branch from 34d7b0d to 0f08eb9 Compare February 25, 2026 12:12
@iscekic
Copy link
Contributor Author

iscekic commented Feb 25, 2026

This seems to be working as intended, closing.

@iscekic iscekic closed this Feb 25, 2026
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.

2 participants