Skip to content
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

Use A Publisher Everwhere #3583

Open
wants to merge 11 commits into
base: master
Choose a base branch
from
Open

Conversation

d80tb7
Copy link
Collaborator

@d80tb7 d80tb7 commented May 13, 2024

Update everywhere that needs to publish Pulsar messages to use a Publisher interface. This means:

  • We're a bit more decoupled from publishes
  • We publish all event messages in a consistent manner. This means we now have a place where we can e.g. log a warning if event messages don't have an optional field filled in.

Chris Martin and others added 11 commits May 13, 2024 18:34
wip
Signed-off-by: Chris Martin <chris@cmartinit.co.uk>
wip
Signed-off-by: Chris Martin <chris@cmartinit.co.uk>
Signed-off-by: Chris Martin <chris@cmartinit.co.uk>
Signed-off-by: Chris Martin <chris@cmartinit.co.uk>
Signed-off-by: Chris Martin <chris@cmartinit.co.uk>
Signed-off-by: Chris Martin <chris@cmartinit.co.uk>
Signed-off-by: Chris Martin <chris@cmartinit.co.uk>
Signed-off-by: Chris Martin <chris@cmartinit.co.uk>
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.

None yet

1 participant