Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The return parameter of send_message was used inconsistently.
This is fixed through this PR.
fixes #31
I benchmarked the code a little with the code given in #142 for different QoS and transport methods.
Using the default TCP with default QoS of 0 seems to be the desirable choice.
Note that the QoS of the subscribe side, is used as a maximum QoS supported.
It is desired to use
set_tcp_nodelay=true
for MQTT brokers to reduce the roundtrip time as noted in #142