<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Sep 8, 2014 at 1:23 PM, Christian Berendt <span dir="ltr"><<a href="mailto:berendt@b1-systems.de" target="_blank">berendt@b1-systems.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">I would like to propose the introduction of a minimum time before<br>
approving a new review request.<br>
<br>
Example: <a href="https://review.openstack.org/#/c/119783/" target="_blank">https://review.openstack.org/#/c/119783/</a><br>
<br>
Uploaded: Sep 8, 2014 4:59 PM<br>
First +2: 6:10 PM<br>
Second +2: 8:15 PM<br>
Approval: 8:16 PM<br>
<br>
I think we should wait at least 24 hours before approving a new review<br>
request (exception: urgent or generated changes). This way everybody of<br>
us has the chance to realize and review new review requests.<br>
<br></blockquote><div><br></div><div>Interesting proposal, we did not address in <a href="https://wiki.openstack.org/wiki/Documentation/ReviewGuidelines">https://wiki.openstack.org/wiki/Documentation/ReviewGuidelines</a> -- my feeling is that we can't all review all patches so it's better to not enforce a minimum wait time. The impatience people have with our reviews should be reversed and we shouldn't have a policy of an enforced wait time.</div><div><br></div><div>To me, it's preferred to have hundreds of patches go through efficiently rather than ensure everyone sees every patch.</div><div><br></div><div>Just my thinking though, would love to hear what others have to say.</div><div><br></div><div>Anne</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
Christian.<br>
<br>
--<br>
Christian Berendt<br>
Cloud Solution Architect<br>
Mail: <a href="mailto:berendt@b1-systems.de">berendt@b1-systems.de</a><br>
<br>
B1 Systems GmbH<br>
Osterfeldstraße 7 / 85088 Vohburg / <a href="http://www.b1-systems.de" target="_blank">http://www.b1-systems.de</a><br>
GF: Ralph Dehner / Unternehmenssitz: Vohburg / AG: Ingolstadt,HRB 3537<br>
<br>
_______________________________________________<br>
Openstack-docs mailing list<br>
<a href="mailto:Openstack-docs@lists.openstack.org">Openstack-docs@lists.openstack.org</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-docs" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-docs</a><br>
</blockquote></div><br></div></div>