<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">2014-04-08 20:08 GMT+08:00 Russell Bryant <span dir="ltr"><<a href="mailto:rbryant@redhat.com" target="_blank">rbryant@redhat.com</a>></span>:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">On 04/08/2014 06:16 AM, Day, Phil wrote:<br>
>> <a href="https://bugs.launchpad.net/nova/+bug/1303983" target="_blank">https://bugs.launchpad.net/nova/+bug/1303983</a><br>
>><br>
>> --<br>
>> Russell Bryant<br>
><br>
> Wow - was there really a need to get that change merged within 12 hours and before others had a chance to review and comment on it ?<br>
<br>
</div>It was targeted against RC2 which we're trying to get out ASAP.  The<br>
change is harmless.<br>
<div class=""><br>
> I see someone has already queried (post the merge) if there isn't a performance impact.<br>
<br>
</div>The commit message indicates that when the API is not used, the<br>
scheduler filters are a no-op.  There is no noticable performance impact.<br></blockquote><div>Thanks Russell, I asked the performance question in the gerrit review. Just checked the logic again and did not found any potential performance issue.  <br>
</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class=""><br>
> I've raised this point before - but apart from non-urgent security fixes shouldn't there be a minimum review period to make sure that all relevant feedback can be given ?<br>
<br>
</div>Separate topic, but no, I do not think there should be any rules on<br>
this.  I think in the majority of cases, people do the right thing.<br>
<br>
In this case, the patch was incredibly trivial and has no performance<br>
impact, so I don't see anything wrong.<br>
<div class="HOEnZb"><div class="h5"><br>
--<br>
Russell Bryant<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>
</div></div></blockquote></div><br><br clear="all"><br>-- <br><div dir="ltr"><div>Thanks,<br><br></div>Jay<br></div>
</div></div>