Thread: [HACKERS] Commitfest 201709 is now closed

[HACKERS] Commitfest 201709 is now closed

From
Daniel Gustafsson
Date:
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

Re: [HACKERS] Commitfest 201709 is now closed

From
Tom Lane
Date:
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

Re: [HACKERS] Commitfest 201709 is now closed

From
Robert Haas
Date:
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

Re: [HACKERS] Commitfest 201709 is now closed

From
Alexander Korotkov
Date:
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.

------
Alexander Korotkov
Postgres Professional: http://www.postgrespro.com
The Russian Postgres Company 

Re: [HACKERS] Commitfest 201709 is now closed

From
Michael Paquier
Date:
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

Re: [HACKERS] Commitfest 201709 is now closed

From
Amit Langote
Date:
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

Re: [HACKERS] Commitfest 201709 is now closed

From
Haribabu Kommi
Date:


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.

How about having an another extra week like (commit week) after commitfest
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