<div class="gmail_quote">On Thu, Feb 21, 2013 at 10:59 AM, Eric Windisch <span dir="ltr"><<a href="mailto:eric@cloudscaling.com" target="_blank">eric@cloudscaling.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

Yes, I'm okay with the duplicate message checking being able to affect<br>
ZMQ, although I still suggest it may be of value to have this be a<br>
toggle as the value is particularly situational. (Generally message<br>
signing *or* mirrored queues - neither are default scenarios)<br></blockquote><div><br>I'm curious.  How do you propose to implement a toggle (a check) say for AMQP mirroring in common code?  Putting implementation specific checks in common code is what concerns me.  We either require all implementations to do duplicate message checking (even future ones) or we move it to implementation specific code.  That is my opinion.<br>
<br>It seems better for the implementation specific code (e.g. AMQP) to check whether some common feature like message singing is enabled and not do duplicate message checking if it were.<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


I have nothing against the AMQP-specific patch<br>
directly, except that I foresee a need to do this in rpc-common for<br>
Havana, and we'll be making a mess for backwards compatibility.</blockquote><div><br>One of my points about duplicate message checking in general is that it is backward and forward compatible.  If a unique ID exists in the message then it is used for duplicate checking.  If a unique ID does not exist, then duplicate checking simply isn't done.  That doesn't sound messy.<br>
</div></div>