<div dir="ltr">So for the point of clarity, this is the behavior of the Java, .net and erlang client libraries -- is it the proper behavior? Should it be changed?<div><br></div><div style>The documentation for RabbitMQ's server configuration does not specify that this is (can be for some clients) the max value.</div>
<div style><br></div><div style>I'm inclined to change it in pika due to the non-documented and non-specified behavior.</div><div style><br></div><div style>Gavin</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">
On Fri, Mar 15, 2013 at 3:48 AM, Simon MacMullen <span dir="ltr"><<a href="mailto:simon@rabbitmq.com" target="_blank">simon@rabbitmq.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Well, the max(client, server) behaviour is also what our clients do so the precedent is there. But ultimately the server will accept whatever the client asks for.<br>
<br>
Cheers, Simon<div class="im"><br>
<br>
On 15/03/13 06:03, Gavin M. Roy wrote:<br>
</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">
In Pika, currently and pre-dating my involvement, when tuning the<br></div>
connection, it uses the /*minimum*/ value above 0 that either the server<div class="im"><br>
or client provides for the heartbeat interval:<br>
<br>
<a href="https://github.com/pika/pika/blob/v0.5/pika/connection.py#L359" target="_blank">https://github.com/pika/pika/<u></u>blob/v0.5/pika/connection.py#<u></u>L359</a><br>
<a href="https://github.com/pika/pika/blob/v0.5/pika/connection.py#L411" target="_blank">https://github.com/pika/pika/<u></u>blob/v0.5/pika/connection.py#<u></u>L411</a><br>
<br>
Since introducing a heartbeat interval sent by the server of 600, this<br></div>
has set a */maximum/* heartbeat interval of 600 regardless of what the<div class="im"><br>
client sets upon creation.<br>
<br>
In reading the 0.9.1 spec, I did not see anything dictating any behavior<br>
of this sort for heartbeat interval negotiation.<br>
<br>
I'm inclined to allow the client to always overwrite the server, but<br>
since this behavior both pre-dates my involvement and is clearly<br>
intentional, I wanted to get some feedback before making the change.<br>
<br>
I currently have an open ticket tied to this that I would like to<br>
resolve before releasing 0.9.10: <a href="https://github.com/pika/pika/issues/303" target="_blank">https://github.com/pika/pika/<u></u>issues/303</a><br>
<br>
So, any thoughts?<br>
<br>
Gavin<br>
<br>
<br>
<br></div>
______________________________<u></u>_________________<br>
rabbitmq-discuss mailing list<br>
<a href="mailto:rabbitmq-discuss@lists.rabbitmq.com" target="_blank">rabbitmq-discuss@lists.<u></u>rabbitmq.com</a><br>
<a href="https://lists.rabbitmq.com/cgi-bin/mailman/listinfo/rabbitmq-discuss" target="_blank">https://lists.rabbitmq.com/<u></u>cgi-bin/mailman/listinfo/<u></u>rabbitmq-discuss</a><br>
<br><span class="HOEnZb"><font color="#888888">
</font></span></blockquote><span class="HOEnZb"><font color="#888888">
<br>
<br>
-- <br>
Simon MacMullen<br>
RabbitMQ, VMware<br>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br>
<p><span style="color:rgb(34,34,34);font-family:arial,sans-serif;font-size:13px;background-color:rgb(255,255,255)">Gavin M. Roy</span><br style="color:rgb(34,34,34);font-family:arial,sans-serif;font-size:13px;background-color:rgb(255,255,255)">
<span style="color:rgb(34,34,34);font-family:arial,sans-serif;font-size:13px;background-color:rgb(255,255,255)">Chief Technology Officer</span><br style="color:rgb(34,34,34);font-family:arial,sans-serif;font-size:13px;background-color:rgb(255,255,255)">
</p><blockquote style="color:rgb(34,34,34);font-family:arial,sans-serif;font-size:13px;background-color:rgb(255,255,255);padding-left:1ex;border-left-color:rgb(204,204,204);border-left-style:solid;margin:0px 0px 0px 0.8ex;border-left-width:1px">
</blockquote><a href="http://www.meetme.com/" style="color:rgb(17,85,204);font-family:arial,sans-serif;font-size:13px;background-color:rgb(255,255,255)" target="_blank"><img src="http://movies.myyearbook.com/emailsignatureimages/logo2.png"></a><br style="color:rgb(34,34,34);font-family:arial,sans-serif;font-size:13px;background-color:rgb(255,255,255)">
<font color="#666666" style="font-family:arial,sans-serif;font-size:13px;background-color:rgb(255,255,255)">100 Union Square Drive<br>New Hope, PA 18938<br>p. +1.215.862.1162 x263<br>f. +1.215.862.0465</font><p></p><p><a href="https://www.facebook.com/pages/MeetMe/21931227129" style="color:rgb(17,85,204);font-family:arial,sans-serif;font-size:13px;background-color:rgb(255,255,255)" target="_blank"><img src="http://movies.myyearbook.com/emailsignatureimages/fb_mini.gif"></a><span style="font-family:arial,sans-serif;font-size:13px;background-color:rgb(255,255,255);color:rgb(136,136,136)"> </span><span style="font-family:arial,sans-serif;font-size:13px;background-color:rgb(255,255,255);color:rgb(136,136,136)"> </span><span style="font-family:arial,sans-serif;font-size:13px;background-color:rgb(255,255,255);color:rgb(136,136,136)"> </span><a href="https://twitter.com/meetme" style="color:rgb(17,85,204);font-family:arial,sans-serif;font-size:13px;background-color:rgb(255,255,255)" target="_blank"><img src="http://movies.myyearbook.com/emailsignatureimages/tw_mini.gif"></a><span style="font-family:arial,sans-serif;font-size:13px;background-color:rgb(255,255,255);color:rgb(136,136,136)"> </span><span style="font-family:arial,sans-serif;font-size:13px;background-color:rgb(255,255,255);color:rgb(136,136,136)"> </span><span style="font-family:arial,sans-serif;font-size:13px;background-color:rgb(255,255,255);color:rgb(136,136,136)"> </span><a href="http://www.youtube.com/user/MeetMeVideos" style="color:rgb(17,85,204);font-family:arial,sans-serif;font-size:13px;background-color:rgb(255,255,255)" target="_blank"><img src="http://movies.myyearbook.com/emailsignatureimages/yt_mini.gif"></a><br style="color:rgb(34,34,34);font-family:arial,sans-serif;font-size:13px;background-color:rgb(255,255,255)">
<font color="#888888" style="font-family:arial,sans-serif;font-size:13px;background-color:rgb(255,255,255)"><br></font><font color="#666666" style="font-family:arial,sans-serif;font-size:13px;background-color:rgb(255,255,255)"><font>The public market leader in social discovery. </font><font>(NYSE MKT: MEET)</font> </font></p>
</div>