From: | Magnus Hagander <magnus(at)hagander(dot)net> |
---|---|
To: | Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com> |
Cc: | pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: commit fest app: Authors |
Date: | 2019-01-04 11:34:58 |
Message-ID: | CABUevExX_ehTxMibddw_FQhxHEavDf38KRe_-wQ-5FBQmR93tg@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Fri, Jan 4, 2019 at 12:26 PM Peter Eisentraut <
peter(dot)eisentraut(at)2ndquadrant(dot)com> wrote:
> Why does the commit fest app not automatically fill in the author for a
> new patch?
>
> And relatedly, every commit fest, there are a few patches registered
> without authors, probably because of the above behavior.
>
> Could this be improved?
>
Can't say I recall why it was done that way originally. I assume you're
suggesting author should default to whomever adds it to the app? Or to
somehow try to match it out of the attached email?
I'm guessing the original discussion could have something to do with a time
when a lot of authors didn't register their own patches (think good old
wiki days), it was just a third party who filled it all out. But today
there is probably no reason not to default it to that, as long as it can be
changed.
I'm not sure it's a good idea to *enforce* it -- because you can't register
an author until they have logged into the CF app once. And I think
registering it with no author is better than registering it with the wrong
author.
--
Magnus Hagander
Me: https://www.hagander.net/ <http://www.hagander.net/>
Work: https://www.redpill-linpro.com/ <http://www.redpill-linpro.com/>
From | Date | Subject | |
---|---|---|---|
Next Message | Peter Eisentraut | 2019-01-04 12:06:24 | Re: Allow auto_explain to log to NOTICE |
Previous Message | Etsuro Fujita | 2019-01-04 11:33:48 | Re: postgres_fdw: oddity in costing aggregate pushdown paths |