Skip to content

fix: properly pass maxPgConnections config through connection chain#589

Merged
jumski merged 1 commit intomainfrom
01-11-fix_pass_maxpgconnections_through_connection_chain
Jan 11, 2026
Merged

fix: properly pass maxPgConnections config through connection chain#589
jumski merged 1 commit intomainfrom
01-11-fix_pass_maxpgconnections_through_connection_chain

Conversation

@jumski
Copy link
Copy Markdown
Contributor

@jumski jumski commented Jan 11, 2026

Fix maxPgConnections Configuration

This PR fixes an issue where the maxPgConnections configuration parameter was being ignored in the edge worker. The parameter is now properly passed through the connection chain with a default value of 4 (previously hardcoded to 10).

Changes:

  • Added maxPgConnections parameter to all relevant interfaces and function calls
  • Updated the connection resolution logic to use the configured value or default to 4
  • Added test functions to verify both default and custom connection limits
  • Added unit tests to validate the connection configuration behavior
  • Added E2E tests to ensure the configuration is properly applied

This change ensures that users can now properly control the maximum number of PostgreSQL connections used by the edge worker, which is important for resource management and performance tuning.

@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Jan 11, 2026

🦋 Changeset detected

Latest commit: c160fb1

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 5 packages
Name Type
@pgflow/edge-worker Patch
@pgflow/core Patch
@pgflow/dsl Patch
@pgflow/client Patch
pgflow Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Copy link
Copy Markdown
Contributor Author

jumski commented Jan 11, 2026

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

@nx-cloud
Copy link
Copy Markdown

nx-cloud bot commented Jan 11, 2026

View your CI Pipeline Execution ↗ for commit c160fb1

Command Status Duration Result
nx run edge-worker:test:integration ✅ Succeeded 3m 53s View ↗
nx run edge-worker:e2e ✅ Succeeded 39s View ↗
nx affected -t verify-exports --base=origin/mai... ✅ Succeeded <1s View ↗
nx affected -t build --configuration=production... ✅ Succeeded <1s View ↗
nx affected -t lint typecheck test --parallel -... ✅ Succeeded 20s View ↗

☁️ Nx Cloud last updated this comment at 2026-01-11 19:07:35 UTC

Copy link
Copy Markdown
Contributor Author

jumski commented Jan 11, 2026

Merge activity

  • Jan 11, 7:20 PM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Jan 11, 7:20 PM UTC: @jumski merged this pull request with Graphite.

@jumski jumski merged commit 2da7657 into main Jan 11, 2026
14 checks passed
@github-actions
Copy link
Copy Markdown
Contributor

🚀 Production Deployment: Website

Successfully deployed to production!

🔗 Production URL: https://pgflow.dev

📝 Details:

  • Commit: 2da7657d961c7e36e579743f1a0c9ea4b1b1c36c
  • View Logs

Deployed at: 2026-01-11T20:20:17+01:00

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