Thread: [HACKERS] Commitfest 201709 is now closed
We have now entered October, which marks the end of Commitfest 201709. All patches have now been dealt with and the commitfest is closed. Before starting on moving, and closing, patches the stat for this commitfest were: Needs review: 69.Waiting on Author: 22.Ready for Committer: 40.Committed: 88.Moved to next CF: 11.Rejected: 8.Returned withFeedback: 18.===========================Total: 256. 29 patches were in need of review but didn’t have a reviewer assigned (and no review posted och -hackers), these have all been moved to the next CF. On top of that, 40 patches were Ready for committer, and have been moved to the next commitfest. The next commitfest is thus quite the smorgasbord for committers already. A few patches had been "Waiting for author" during the entire commitfest without seeing any updates, and were thus closed as “Returned with feddback”. A few others patches were Returned with feedback too, due to either not moving from Waiting for author, or due to review comments being made. And last, but not least, a lot of patches were pushed to the next commitfest. What I’ve tried to do is move patches such that every patch submitted has a fair chance at a good review. Also, patches being actively discussed and worked on were moved of course. With the volumes in mind, I’m absolutely certain I’ve botched one or two up though. If you have a patch that was moved, and you intend to rewrite enough of it to warrant a resubmission then please go in and close your entry. In summary, this commitfest saw a large number of patches, and while lots of patches got reviewed and committed, there is a lot of improvement to be had when it comes to closing them. We clearly need more bandwidth among reviewers and committers to cope with the increasing size of the commitfests. There is a clear risk that too much gets moved to the next commitfest making each new commitfest progressively larger and harder to handle. I don’t have any good answers, but we probably need to think of something going forward. Thanks to everyone who participated, and to everyone who have responded to my nagging via the CF app email function. This is clearly an awesome community. cheers ./daniel -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Daniel Gustafsson <daniel@yesql.se> writes: > Thanks to everyone who participated, and to everyone who have responded to my > nagging via the CF app email function. This is clearly an awesome community. And thanks to you for your hard work as CFM! That's tedious and largely thankless work, but it's needed to keep things moving. regards, tom lane -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
On Mon, Oct 2, 2017 at 11:57 AM, Tom Lane <tgl@sss.pgh.pa.us> wrote: > Daniel Gustafsson <daniel@yesql.se> writes: >> Thanks to everyone who participated, and to everyone who have responded to my >> nagging via the CF app email function. This is clearly an awesome community. > > And thanks to you for your hard work as CFM! That's tedious and > largely thankless work, but it's needed to keep things moving. +1. I think we need to figure out some plan for tackling the backlog of Ready for Committer patches, though. There are currently 42 of them, 10 of which are listed (maybe not all correctly) as bug fixes. That's not great. -- Robert Haas EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
On Mon, Oct 2, 2017 at 6:57 PM, Tom Lane <tgl@sss.pgh.pa.us> wrote:
Daniel Gustafsson <daniel@yesql.se> writes:
> Thanks to everyone who participated, and to everyone who have responded to my
> nagging via the CF app email function. This is clearly an awesome community.
And thanks to you for your hard work as CFM! That's tedious and
largely thankless work, but it's needed to keep things moving.
+1,
Thank you very much, Daniel! It was a pleasure working with you at commitfest.
The Russian Postgres Company
On Tue, Oct 3, 2017 at 1:23 AM, Alexander Korotkov <a.korotkov@postgrespro.ru> wrote: > On Mon, Oct 2, 2017 at 6:57 PM, Tom Lane <tgl@sss.pgh.pa.us> wrote: >> >> Daniel Gustafsson <daniel@yesql.se> writes: >> > Thanks to everyone who participated, and to everyone who have responded >> > to my >> > nagging via the CF app email function. This is clearly an awesome >> > community. >> >> And thanks to you for your hard work as CFM! That's tedious and >> largely thankless work, but it's needed to keep things moving. > > +1, > Thank you very much, Daniel! It was a pleasure working with you at > commitfest. Thanks for doing a bunch of work, Daniel. This is a difficult task. -- Michael -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
On 2017/10/03 7:16, Michael Paquier wrote: > On Tue, Oct 3, 2017 at 1:23 AM, Alexander Korotkov > <a.korotkov@postgrespro.ru> wrote: >> On Mon, Oct 2, 2017 at 6:57 PM, Tom Lane <tgl@sss.pgh.pa.us> wrote: >>> >>> Daniel Gustafsson <daniel@yesql.se> writes: >>>> Thanks to everyone who participated, and to everyone who have responded >>>> to my >>>> nagging via the CF app email function. This is clearly an awesome >>>> community. >>> >>> And thanks to you for your hard work as CFM! That's tedious and >>> largely thankless work, but it's needed to keep things moving. >> >> +1, >> Thank you very much, Daniel! It was a pleasure working with you at >> commitfest. > > Thanks for doing a bunch of work, Daniel. This is a difficult task. +1. Thank you, Daniel! Regards, Amit -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
On Tue, Oct 3, 2017 at 3:12 AM, Robert Haas <robertmhaas@gmail.com> wrote:
On Mon, Oct 2, 2017 at 11:57 AM, Tom Lane <tgl@sss.pgh.pa.us> wrote:
> Daniel Gustafsson <daniel@yesql.se> writes:
>> Thanks to everyone who participated, and to everyone who have responded to my
>> nagging via the CF app email function. This is clearly an awesome community.
>
> And thanks to you for your hard work as CFM! That's tedious and
> largely thankless work, but it's needed to keep things moving.
+1.
I think we need to figure out some plan for tackling the backlog of
Ready for Committer patches, though. There are currently 42 of them,
10 of which are listed (maybe not all correctly) as bug fixes. That's
not great.
closed(may be new state like "commit" or etc) for the Ready for committer
patches to let have the feed back from committers, In case of some rework
is required and it will be get ready by the next commitfest begins.
I feel something along these lines may get earlier feedback to the the patches.
Regards,
Hari Babu
Fujitsu Australia