The last patch set of this change has been in code review for almost two weeks with the only code review comments being to remove a method from the unit tests that I used for debugging but left in with the call commented out.  This patch set removes that uncalled method.<br>
<br>I don't understand why this RPC code change that provides a 23X RPC call maximum throughput improvement to a cluster of 3 RabbitMQ servers with mirroring enabled, would not get approved?  It takes the maximum throughput from 35 RPC calls/sec to 800 RPC calls/sec.<br>
<br>Do we really want to wait until Havana for this fix?<br>
<br>Thanks,<br>Ray<br><br><div class="gmail_quote">On Tue, Feb 12, 2013 at 11:19 PM, Jenkins (Code Review) <span dir="ltr"><<a href="mailto:review@openstack.org" target="_blank">review@openstack.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Jenkins has posted comments on this change.<br>
<div class="im"><br>
Change subject: blueprint amqp-rpc-fast-reply-queue<br>
......................................................................<br>
<br>
<br>
</div>Patch Set 13: Works for me<br>
<br>
Build succeeded.<br>
<br>
- <a href="http://logs.openstack.org/19721/13/check/gate-oslo-incubator-pyflakes/697" target="_blank">http://logs.openstack.org/19721/13/check/gate-oslo-incubator-pyflakes/697</a> : SUCCESS<br>
- <a href="http://logs.openstack.org/19721/13/check/gate-oslo-incubator-docs/807" target="_blank">http://logs.openstack.org/19721/13/check/gate-oslo-incubator-docs/807</a> : SUCCESS<br>
- <a href="http://logs.openstack.org/19721/13/check/gate-oslo-incubator-pep8/993" target="_blank">http://logs.openstack.org/19721/13/check/gate-oslo-incubator-pep8/993</a> : SUCCESS<br>
- <a href="http://logs.openstack.org/19721/13/check/gate-oslo-incubator-python26/993" target="_blank">http://logs.openstack.org/19721/13/check/gate-oslo-incubator-python26/993</a> : SUCCESS<br>
- <a href="http://logs.openstack.org/19721/13/check/gate-oslo-incubator-python27/993" target="_blank">http://logs.openstack.org/19721/13/check/gate-oslo-incubator-python27/993</a> : SUCCESS<br>
<div class="im"><br>
--<br>
To view, visit <a href="https://review.openstack.org/19721" target="_blank">https://review.openstack.org/19721</a><br>
To unsubscribe, visit <a href="https://review.openstack.org/settings" target="_blank">https://review.openstack.org/settings</a><br>
<br>
</div>Gerrit-MessageType: comment<br>
Gerrit-Change-Id: Idb09a71472866bd3950f58d4f7f45a3181eb40fc<br>
Gerrit-PatchSet: 13<br>
<div class="im">Gerrit-Project: openstack/oslo-incubator<br>
Gerrit-Branch: master<br>
Gerrit-Owner: Raymond Pekowski <<a href="mailto:pekowski@gmail.com">pekowski@gmail.com</a>><br>
Gerrit-Reviewer: Andrea Rosa <<a href="mailto:andrea.rosa@hp.com">andrea.rosa@hp.com</a>><br>
Gerrit-Reviewer: Chris Behrens <<a href="mailto:cbehrens%2Bgerrit@codestud.com">cbehrens+gerrit@codestud.com</a>><br>
</div>Gerrit-Reviewer: Davanum Srinivas (dims) <<a href="mailto:davanum@gmail.com">davanum@gmail.com</a>><br>
<div class="HOEnZb"><div class="h5">Gerrit-Reviewer: Eric Windisch <<a href="mailto:eric@cloudscaling.com">eric@cloudscaling.com</a>><br>
Gerrit-Reviewer: Jenkins<br>
Gerrit-Reviewer: Joshua Harlow <<a href="mailto:harlowja@yahoo-inc.com">harlowja@yahoo-inc.com</a>><br>
Gerrit-Reviewer: Raymond Pekowski <<a href="mailto:pekowski@gmail.com">pekowski@gmail.com</a>><br>
Gerrit-Reviewer: Russell Bryant <<a href="mailto:rbryant@redhat.com">rbryant@redhat.com</a>><br>
</div></div></blockquote></div><br>