Обсуждение: Commit fest 2016-09 is now closed
Hi all, Per $subject, I would like to thank on behalf of Fabrizio everybody who has taken the time to send patches, review them and argue about them. This has been by experience the largest commit fest ever, with more than 200 entries in total. The delay in stabilizing 9.6 is likely the explanation behind such a high number, a lot of patches have been parked there for a couple of months. There was something like 60~70 patches still listed as active in the CF app on Friday my time, so doing the vacuum cleanup has taken some time by marking patches as returned with feedback, rejected (few) and moved to next CF. A couple of things I noted when going through the remaining entries: - a lot of folks have registered to review a patch and did not post a review. Please avoid putting your name on a patch if you can't afford the time to look at a patch. - The rule one patch sent = one patch review of equal difficulty still works. - Be careful of the status of the patch, and update it correctly. I have noticed a lot of improvements in this area actually! Here is the final score of this session: - Committed: 103. - Moved to next CF: 51. - Rejected: 12. - Returned with Feedback: 53. Total: 219. Thanks, -- Michael
On Mon, Oct 3, 2016 at 2:01 PM, Michael Paquier <michael.paquier@gmail.com> wrote: > There was something like 60~70 patches still listed as active in the > CF app on Friday my time, so doing the vacuum cleanup has taken some > time by marking patches as returned with feedback, rejected (few) and > moved to next CF. A couple of things I noted when going through the > remaining entries: > - a lot of folks have registered to review a patch and did not post a > review. Please avoid putting your name on a patch if you can't afford > the time to look at a patch. > - The rule one patch sent = one patch review of equal difficulty still works. > - Be careful of the status of the patch, and update it correctly. I > have noticed a lot of improvements in this area actually! Forgot something: if you feel that your patch did not receive the correct treatment, of course feel free to update its status in the CF app, feel free as well to complain at me and/or on this thread if necessary. I am sure we will be able to sort things out :) -- Michael
<div dir="ltr"><div class="gmail_extra"><br />On Mon, Oct 3, 2016 at 2:05 AM, Michael Paquier <<a href="mailto:michael.paquier@gmail.com">michael.paquier@gmail.com</a>>wrote:<br />><br />> On Mon, Oct 3, 2016 at2:01 PM, Michael Paquier<br />> <<a href="mailto:michael.paquier@gmail.com">michael.paquier@gmail.com</a>> wrote:<br/>> > There was something like 60~70 patches still listed as active in the<br />> > CF app on Fridaymy time, so doing the vacuum cleanup has taken some<br />> > time by marking patches as returned with feedback,rejected (few) and<br />> > moved to next CF. A couple of things I noted when going through the<br />>> remaining entries:<br />> > - a lot of folks have registered to review a patch and did not post a<br />>> review. Please avoid putting your name on a patch if you can't afford<br />> > the time to look at a patch.<br/>> > - The rule one patch sent = one patch review of equal difficulty still works.<br />> > - Be carefulof the status of the patch, and update it correctly. I<br />> > have noticed a lot of improvements in this areaactually!<br />><br />> Forgot something: if you feel that your patch did not receive the<br />> correct treatment,of course feel free to update its status in the CF<br />> app, feel free as well to complain at me and/or onthis thread if<br />> necessary. I am sure we will be able to sort things out :)<br /><br /></div><div class="gmail_extra">HiMichael,<br /><br /></div><div class="gmail_extra">Thank you so much for your help. I was travellingto take a rest after a long and busy month. <br /><br /></div><div class="gmail_extra">I'll update my statisticsand send another email to check the % of the work was done.<br /></div><div class="gmail_extra"><br /></div><divclass="gmail_extra">Thanks again,<br /></div><div class="gmail_extra"><br />--<br />Fabrízio de Royes Mello<br/>Consultoria/Coaching PostgreSQL<br />>> Timbira: <a href="http://www.timbira.com.br">http://www.timbira.com.br</a><br/>>> Blog: <a href="http://fabriziomello.github.io">http://fabriziomello.github.io</a><br/>>> Linkedin: <a href="http://br.linkedin.com/in/fabriziomello">http://br.linkedin.com/in/fabriziomello</a><br/>>> Twitter: <a href="http://twitter.com/fabriziomello">http://twitter.com/fabriziomello</a><br/>>> Github: <a href="http://github.com/fabriziomello">http://github.com/fabriziomello</a></div></div>