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
Add pubsub support for kafka #315
Conversation
Change looks good, @khous. Have you tested this in any way? |
Yes I have. Here's the integration test I'm relying on I also used one of the sample projects to manually test the functionality. I swapped out redis in the 4.pub-sub sample |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Please finish off by updating the docs here: https://github.com/dapr/docs/tree/master/howto/setup-pub-sub-message-broker |
Description
Add kafka binding for pubsub
Issue reference
#8
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:
I have an e2e test which runs just the pubsub functions against kafka. Need to figure out the best way to integrate this. The current e2e tests don't appear to support testing multiple pubsub components, but I might be mistaken.