<div dir="ltr">Honestly I have been doing this for some time based on the criticality of the patches and of course with an agreement with the original author of the patches. Another factor we need to consider here about patches where the comments were available and haven&#39;t been addressed for a significant period. We should also need to consider such patches based on their importance and refresh them. <br><div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Oct 31, 2017 at 4:21 PM, Jeff Darcy <span dir="ltr">&lt;<a href="mailto:jeff@pl.atyp.us" target="_blank">jeff@pl.atyp.us</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><u></u>




<div><span class=""><div style="font-family:Arial"><br></div>
<div><br></div>
<div><br></div>
<div>On Tue, Oct 31, 2017, at 01:01 AM, Amar Tumballi wrote:<br></div>
<blockquote type="cite"><div dir="ltr"><div>In this case, I suggest maintainers can send a message to author, and send an updated patch with their suggestion (with making sure &#39;--author&#39; is set to original author). This can save both the effort of review, and also heart burn of someone not understanding the comments properly. <br></div>
</div>
</blockquote><div style="font-family:Arial"><br></div>
</span><div style="font-family:Arial">I would really like it if we could get to the point where maintainers (or others) could feel comfortable updating other contributors&#39; patches, because it really would improve our development velocity.  I&#39;ve done it very sparingly, usually only for patches that the author seemed to have given up on, because there is a risk of people being offended.  It can feel like someone else is trying to take control of - or even credit for - one&#39;s own work.  To avoid this, I think we need to do two things:<br></div>
<div style="font-family:Arial"><br></div>
<div style="font-family:Arial">(1) Thoroughly document how to update someone else&#39;s patch while retaining proper credit for their work, and how to accept such an update into one&#39;s own local repository.  This addresses the technical/logistical issue.<br></div>
<div style="font-family:Arial"><br></div>
<div style="font-family:Arial">(2) Recognize new contributors as such and automatically (or at least semi-automatically) send them email explaining our expectations and standards for review etiquette - including this, but other things as well.  This addresses the cultural issue.</div>
</div>

<br>______________________________<wbr>_________________<br>
maintainers mailing list<br>
<a href="mailto:maintainers@gluster.org">maintainers@gluster.org</a><br>
<a href="http://lists.gluster.org/mailman/listinfo/maintainers" rel="noreferrer" target="_blank">http://lists.gluster.org/<wbr>mailman/listinfo/maintainers</a><br>
<br></blockquote></div><br></div></div></div>