Hi Richard, > Pushed. > > I still listed myself as Co-authored-by rather than Author, because I > want to give full credit to you, as this is your idea and your patch.
Thank you for the commit! I also appreciate your consideration in giving me full credit for the patch. Regards, Tatsuro Yamada > -----Original Message----- > From: Richard Guo <[email protected]> > Sent: Thursday, July 9, 2026 10:02 AM > To: Tatsuro Yamada（山田達朗） <[email protected]> > Cc: Tatsuro Yamada <[email protected]>; David Rowley > <[email protected]>; Ashutosh Bapat > <[email protected]>; [email protected] > Subject: Re: Add enable_groupagg GUC parameter to control GroupAggregate > usage > > On Wed, Jul 8, 2026 at 8:54 PM Tatsuro Yamada <[email protected]> > wrote: > > I'm in favor of committing this. > > Before you do, I'd like to clarify one point regarding the commit message. > > Would it make sense to list both of us as authors? > > > > The reason I ask is that you made substantial changes to the patch, > > including handling additional plan nodes such as SetOp and adding > regression > > tests. Given those contributions, I wondered whether it would be more > > appropriate for both of us to be listed as authors. (I've seen past commits > that > > included two Author: lines.) > > I'm happy to leave that decision to you. > > Pushed. > > I still listed myself as Co-authored-by rather than Author, because I > want to give full credit to you, as this is your idea and your patch. > > - Richard
