<p dir="ltr">Quickly replying on list since I'm having trouble using gerrit on my phone...</p>
<p dir="ltr">-1; The tests need to come out of common since these tests are used by the zeromq tests, which will break. Otherwise, it looks fairly reasonable at a glance, although I would like to evaluate it a bit further. As others indicated, there might be ways to improve backwards compat. </p>
<p dir="ltr">As an aside, we're already using some similar patterns in the zeromq driver. There might be further room for reuse here, but as I said, I'll review the patch in more depth when I'm back at a keyboard.</p>
<div class="gmail_quote">On Jan 21, 2013 8:32 AM, "Ray Pekowski" <<a href="mailto:pekowski@gmail.com">pekowski@gmail.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="gmail_quote">On Thu, Jan 17, 2013 at 6:24 PM, Ray Pekowski <span dir="ltr"><<a href="mailto:pekowski@gmail.com" target="_blank">pekowski@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br><div class="gmail_quote"><div></div><div>I got approval to share the results of my scalability study of RabbitMQ
as used by OpentStack done as part of my work at Dell. It shows the
improvement that comes from the single response queue change proposed by the blueprint. Here is a link:<br><br><a href="https://docs.google.com/file/d/0B-droFdkDaVhVzhsN3RKRlFLODQ/edit" target="_blank">https://docs.google.com/file/d/0B-droFdkDaVhVzhsN3RKRlFLODQ/edit</a> <br>
<span><font color="#888888">
</font></span></div></div><span><font color="#888888">
</font></span></blockquote></div><br>For those not having enough time to go over the Dell presentation at the above link, the short summary is the amqp-rpc-fast-reply-queue blueprint provides a 23 times improvement to RPC call over a cluster of three
RabbitMQ servers with mirroring enabled. The RPC call rate goes from 35 RPC calls/sec to
812 RPC calls/sec.<br><br>The blueprint can be found here:<br><a href="https://blueprints.launchpad.net/oslo/+spec/amqp-rpc-fast-reply-queue" target="_blank">https://blueprints.launchpad.net/oslo/+spec/amqp-rpc-fast-reply-queue</a><br>
<br>
The code review here:<br><a rel="nofollow" href="https://review.openstack.org/#/c/19721/" target="_blank">https://review.openstack.org/#/c/19721/</a><br><br>For those interested, please review the code so that it can be approved.<br>
<br>Thanks,<br>Ray<br><br>
<br>_______________________________________________<br>
OpenStack-dev mailing list<br>
<a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
<br></blockquote></div>