Skip to content

Conversation

@edgurgel
Copy link
Member

What kind of change does this PR introduce?

We can do this because we know all subscriptions for this tenant that should receive a message

This way we only increment in a single place ensuring that we are respecting the limits and counting them properly.

Additional context

Add any other context or screenshots.

@edgurgel edgurgel force-pushed the fix/cdc-rls-rate-counter branch from 48b9980 to c73c7c3 Compare October 20, 2025 22:25
@coveralls
Copy link

coveralls commented Oct 20, 2025

Coverage Status

coverage: 87.079% (+0.3%) from 86.757%
when pulling 72862a9 on fix/cdc-rls-rate-counter
into 8e4d3e6 on main.

We can do this because we know all subscriptions for this tenant that
should receive a message
@edgurgel edgurgel force-pushed the fix/cdc-rls-rate-counter branch from c73c7c3 to 72862a9 Compare October 31, 2025 01:15
@edgurgel edgurgel merged commit d7bacb8 into main Oct 31, 2025
5 of 6 checks passed
@edgurgel edgurgel deleted the fix/cdc-rls-rate-counter branch October 31, 2025 01:26
@kiwicopple
Copy link
Member

🎉 This PR is included in version 2.58.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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