<div dir="ltr">Hi Ulf,<div>great work !</div><div><br></div><div>Just few considerations on the MQTT LWT service.</div><div>As I mentioned last time, the LWT feature is not a really "MQTT related" but is having it on top of AMQP.</div><div><br></div><div>In the mqtt-lwt project I see :</div><div><ul><li>the Netty MQTT coded dependency is used just for the MQTT QoS enum, that could be removed (using an "ad hoc" enum or just 0,1 or 2) -> no dependencies on MQTT related projects anymore</li><li>the only two AmqpWillMessage and AmqpPublishMessage classes involved use standard AMQP but add two annotations "x-opt-mqtt-qos" and "x-opt-retain-message". The first one could be not related to MQTT so we could remove the "-mqtt-" and in part it's decoded using the durable AMQP field (<a href="https://github.com/EnMasseProject/enmasse/blob/master/documentation/mqtt-over-amqp/docs/operations_connection.md#will-information-to-the-last-will-and-testament-service">https://github.com/EnMasseProject/enmasse/blob/master/documentation/mqtt-over-amqp/docs/operations_connection.md#will-information-to-the-last-will-and-testament-service</a>). The retain part is more MQTT specific (for the will message) even if you could not specify it having retain=false by default.</li></ul><div>My instinct (but more thinking is needed) is to have an amqp-lwt service, so outside of the whole "mqtt" umbrella.</div></div><div><br></div><div>What do you and the other guys think about that ?</div><div><br></div><div>In any case, today there is some messages classes duplicated between mqtt-gateway and mqtt-lwt but now that they are under the same repo is simpler to have an mqtt-common for them :-)</div><div><br></div><div>Thanks,</div><div>Paolo</div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jul 4, 2017 at 6:19 AM, Ulf Lilleengen <span dir="ltr"><<a href="mailto:ulilleen@redhat.com" target="_blank">ulilleen@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi all,<br>
<br>
The EnMasse repositories[1] have now been merged into:<br>
<br>
<a href="https://github.com/EnMasseProject/enmasse" rel="noreferrer" target="_blank">https://github.com/EnMasseProj<wbr>ect/enmasse</a><br>
<br>
All development on the former repositories will now be moved to the new one. See <a href="https://github.com/EnMasseProject/enmasse/blob/master/HACKING.md" rel="noreferrer" target="_blank">https://github.com/EnMasseProj<wbr>ect/enmasse/blob/master/<wbr>HACKING.md</a> for instructions on building etc.<br>
<br>
Github issues, this mailing list or the #enmasse channel on freenode are good places to ask should you be having issues building.<br>
<br>
The old repositories have been marked with a 'moved' notification in their respective README.md files, but will be kept around for some time just in case.<br>
<br>
Repositories that doesn't directly involve EnMasse and have an independent release schedule will remain in their own repositories.<br>
<br>
[1] Repositories moved: admin, mqtt-gateway, mqtt-lwt, ragent, subserv, routilities, router-image, artemis-image, topic-forwarder, systemtests, router-metrics<br>
<br>
Happy hacking,<br>
<br>
Ulf<br>
<br>
______________________________<wbr>_________________<br>
enmasse mailing list<br>
<a href="mailto:enmasse@redhat.com" target="_blank">enmasse@redhat.com</a><br>
<a href="https://www.redhat.com/mailman/listinfo/enmasse" rel="noreferrer" target="_blank">https://www.redhat.com/mailman<wbr>/listinfo/enmasse</a><br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><p style="color:rgb(0,0,0);font-family:overpass,sans-serif;font-weight:bold;margin:0px;padding:0px;font-size:14px;text-transform:uppercase"><span>PAOLO</span> <span>PATIERNO</span></p><p style="color:rgb(0,0,0);font-family:overpass,sans-serif;font-size:10px;margin:0px 0px 4px;text-transform:uppercase"><span>SENIOR SOFTWARE ENGINEER, MESSAGING & IOT</span></p><p style="font-family:overpass,sans-serif;margin:0px;font-size:10px;color:rgb(153,153,153)"><a href="https://www.redhat.com/" style="color:rgb(0,136,206);margin:0px;text-decoration:none" target="_blank">Red Hat <span><br><br></span></a></p><table border="0" style="color:rgb(0,0,0);font-family:overpass,sans-serif;font-size:medium"><tbody><tr><td width="100px"><a href="https://red.ht/sig" target="_blank"><img src="https://www.redhat.com/profiles/rh/themes/redhatdotcom/img/logo-red-hat-black.png" width="90" height="auto"></a></td><td style="font-size:10px"><div><a href="https://redhat.com/summit" style="text-decoration:none" target="_blank"><img width="70px" height="auto" src="https://www.redhat.com/cms/managed-files/summit_logo_170_0.png" style="outline:0px"></a></div></td></tr></tbody></table></div></div>
</div>