Skip to content

Optimize Database Connection Pooling - #596

Merged
gelluisaac merged 1 commit into
Traqora:mainfrom
Douglas4tech:Database
Jul 27, 2026
Merged

Optimize Database Connection Pooling#596
gelluisaac merged 1 commit into
Traqora:mainfrom
Douglas4tech:Database

Conversation

@Douglas4tech

@Douglas4tech Douglas4tech commented Jul 27, 2026

Copy link
Copy Markdown
Contributor

Description

Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context.

Fixes #544
Fixes #545
Fixes #477
Fixes #478

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

Size

Keep pull requests reviewable: ≤ 1000 lines changed and ≤ 10 files changed.
An automated check comments (but never fails CI) when a PR goes over.

  • This PR is within the size limits, or it is a planned large refactor
    labelled refactor:large (≤ 50 files), or the title contains
    [large PR] with a justification in the description.

If it is too big: split it into a stack of smaller PRs, land refactors separately
from behaviour changes, use a feature flag to merge incomplete work
incrementally, and isolate generated/vendored files. See
docs/PR_SIZE_LIMITS.md.

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream modules

@github-actions

Copy link
Copy Markdown

📏 Pull request size warning

This pull request exceeds the review-friendly size limits. CI is not blocked — this is guidance, not a gate.

Metric This PR Limit
Lines changed 6429 1000
Files changed 30 10

Why it matters

Review quality drops sharply with diff size: defects slip through, review turnaround grows, and reverts become riskier because unrelated changes are bundled together.

What to do

  • Split the change into a stack of smaller, independently reviewable PRs.
  • Land refactors and behaviour changes separately.
  • Hide incomplete work behind a feature flag and merge it incrementally.
  • Move generated files, lockfiles, or vendored code into their own PR.

Legitimately large?

  • Add the refactor:large label to raise the file ceiling to 50 files.
  • Add [large PR] to the title to skip this check entirely.
Breached limits
  • 6429 lines changed (+6386/-43) exceeds the 1000-line limit.
  • 30 files changed exceeds the 10-file limit.

@drips-wave

drips-wave Bot commented Jul 27, 2026

Copy link
Copy Markdown

@Douglas4tech Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@gelluisaac
gelluisaac merged commit 4a3a5b4 into Traqora:main Jul 27, 2026
7 of 28 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

3 participants