Skip to content

WPB-17378 gundeck: Fix logic to split pushes between cassandra and rabbitmq #4556

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.

Sign up for GitHub

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

Jump to bottom
Merged
akshaymankar merged 6 commits into from
Apr 29, 2025
Merged

WPB-17378 gundeck: Fix logic to split pushes between cassandra and rabbitmq #4556

akshaymankar merged 6 commits into from
Apr 29, 2025

Conversation

@battermann
Copy link
Contributor

@battermann battermann commented Apr 25, 2025 •
edited
Loading

https://wearezeta.atlassian.net/browse/WPB-17378

Checklist

  • Add a new entry in an appropriate subdirectory of changelog.d
  • Read and follow the PR guidelines

@zebot zebot added the ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist label Apr 25, 2025
@akshaymankar akshaymankar force-pushed the WPB-17378-be-filter-out-messages-from-events-which-are-distributed-back-to-their-sender-by-be branch from 8e63b90 to 015b8b3 Compare April 28, 2025 12:42
@battermann
introduce new RecipientClients constructor
43024e0 
- express the only temporary queue recipient case
- property test
@battermann battermann force-pushed the WPB-17378-be-filter-out-messages-from-events-which-are-distributed-back-to-their-sender-by-be branch from 015b8b3 to 43024e0 Compare April 28, 2025 14:23
@battermann battermann marked this pull request as ready for review April 28, 2025 15:35
@battermann battermann requested review from a team as code owners April 28, 2025 15:35
@akshaymankar akshaymankar changed the title WPB-17378 filter out messages from /events which are distributed back to their sender by BE WPB-17378 gundeck: Fix logic to split pushes between cassandra and rabbitmq Apr 29, 2025
akshaymankar
akshaymankar reviewed Apr 29, 2025
View reviewed changes
@akshaymankar akshaymankar merged commit 206b4ff into develop Apr 29, 2025
8 checks passed
@akshaymankar akshaymankar deleted the WPB-17378-be-filter-out-messages-from-events-which-are-distributed-back-to-their-sender-by-be branch April 29, 2025 09:37
This was referenced May 12, 2025
Closed
Closed
Closed
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@akshaymankar akshaymankar akshaymankar approved these changes

Assignees

No one assigned

Labels

ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@battermann @akshaymankar @zebot