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

MQTT Initial Implementation #25

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

Conversation

ahmadshahwan
Copy link

No description provided.

Ahmad Shahwan added 6 commits September 17, 2015 12:19
Using Eclipse Paho MQTT client.
This allow peers to get notified when onther peer loses its MQTT client.
Allowing them to switch to another protocol if available.
@drem-darios
Copy link

👍 Are you working on a Java based MQTT impl? If so I can contribute to that fork.

@ahmadshahwan
Copy link
Author

Hello Drem, indeed we have Java implementation in mind, but no much work has been done in this direction yet. So you're most welcome to kick it off.
A generic "how to implement a new protocol for Herald" is available on the wiki. It's more meant for Python (I wrote it while implementing MQTT in Python) but it should give good indications for Java as well.

@drem-darios
Copy link

Great! I'll see if I get something working in my spare time. I'm assuming there will need to be other work done in a different cohorte project to get configuration working. Are you working on the mqtt configuration for python?

@ahmadshahwan
Copy link
Author

Ideally, Herald is designed to allow "pluggable" new transports (such as HTTP, XMPP, MQTT) without even the need to change other files (including those of the same project).
In Java, and using OSGi, it boils down to implementing (and providing) two services:

  • org.cohorte.herald.ITransport, and
  • org.cohorte.herald.ITransportDirectory.

Unfortunately, in my commits on this branch I "mistakenly" did some changes that are not related to the MQTT implementation (fixing a minor bug, reformatting code). Relevant changes are only files added under the python/herald/transports/mqtt/ directory.

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.

2 participants