Skip to content

fix: region rebalancing for Postgres Changes #1464

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

Merged
merged 1 commit into from
Jul 22, 2025

Conversation

edgurgel
Copy link
Member

What kind of change does this PR introduce?

Follow-up from #1455 to also rebalance our Postgres Changes pool of processes to the correct region

What is the current behavior?

If the Postgres Changes pool of processes start in the wrong region they might have a much higher latency that can impact performance.

What is the new behavior?

Every X time it will check if it's running at the correct region. If it's not then it stops. The next time it runs it will start at the right place.

Additional context

Add any other context or screenshots.

Copy link

vercel bot commented Jul 16, 2025

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

1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
realtime-demo ⬜️ Ignored (Inspect) Visit Preview Jul 22, 2025 9:31am

@coveralls
Copy link

coveralls commented Jul 16, 2025

Coverage Status

coverage: 85.063% (-0.2%) from 85.258%
when pulling 7752e54 on fix/subscription-manager-rebalance
into 119cd54 on main.

@edgurgel edgurgel force-pushed the fix/subscription-manager-rebalance branch from 90f7c3f to 7752e54 Compare July 22, 2025 09:31
@edgurgel edgurgel merged commit 1efdf78 into main Jul 22, 2025
5 checks passed
@edgurgel edgurgel deleted the fix/subscription-manager-rebalance branch July 22, 2025 09:43
@kiwicopple
Copy link
Member

🎉 This PR is included in version 2.41.5 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Copy link

@kimsoo527 kimsoo527 left a comment

Choose a reason for hiding this comment

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

Incredible performance

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants