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

adding scripts for invocating MQTT sensors #77

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mailmahee
Copy link

adding scripts to invoke mqtt sensors to publish messages to kafka topics.

@mailmahee
Copy link
Author

mailmahee commented Dec 13, 2018

@gnumoreno @eyalgutkind @tzach - please review and merge.

@tzach
Copy link
Collaborator

tzach commented Dec 13, 2018

Looks good to me
Are you planing to add Scylla to this system?
Please add running instructions to the README

@mailmahee
Copy link
Author

@tzach this is part of a script which when run will generate MQTT messages that get published to Kafka - There are a set of instructions for spinning up Kafka connect and then simply connecting it to Scylla - all the instructions are captured in a blog we are planning on publishing.
https://docs.google.com/document/d/1DnVf-rVjMank5_rLQv1y8gucpPxvnX3CdFu9MLEuY60/edit?ts=5c116437

@tzach
Copy link
Collaborator

tzach commented Dec 16, 2018

Thanks @mailmahee
@eyalgutkind @gnumoreno can you help review the code? (or at least validate its working)

@mailmahee
Copy link
Author

@tzach - can you just go ahead merge it - have waited for more than a few days now...

@mailmahee
Copy link
Author

@PeterCorless is blocked on this

@eyalgutkind
Copy link

@mailmahee just take the documentation of the usage from the blog and use it as the README.md

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

Successfully merging this pull request may close these issues.

4 participants