public inbox for [email protected]  
help / color / mirror / Atom feed
From: Daniel Gustafsson <[email protected]>
To: Bruce Momjian <[email protected]>
Cc: David G. Johnston <[email protected]>
Cc: Álvaro Herrera <[email protected]>
Cc: Peter Geoghegan <[email protected]>
Cc: Andrey Borodin <[email protected]>
Cc: PostgreSQL-development <[email protected]>
Subject: Re: PG 19 release notes and authors
Date: Mon, 6 Apr 2026 16:03:27 +0200
Message-ID: <[email protected]> (raw)
In-Reply-To: <[email protected]>
References: <[email protected]>
	<[email protected]>
	<CAKFQuwY8Fm_JdXQUvEX5KGXbfLh3onZeFj9ZPEL9aB88wT7aQA@mail.gmail.com>
	<[email protected]>

> On 6 Apr 2026, at 15:57, Bruce Momjian <[email protected]> wrote:

> Wow, I never thought that was a valid pattern, but I see a few PG 19
> commit messages using that, e.g.:
> 
> Author: Peter Eisentraut <[email protected]>
> 2025-08-12 [5f19d13df] libpq: Set LDAP protocol version 3
> 
>     libpq: Set LDAP protocol version 3
> 
>     Some LDAP servers reject the default version 2 protocol.  So set
>     version 3 before starting the connection.  This matches how the
>     backend LDAP code has worked all along.
> 
>     Co-authored-by: Andrew Jackson <[email protected]>
>     Reviewed-by: Pavel Seleznev <[email protected]>
>     Discussion: https://www.postgresql.org/message-id/flat/CAKK5BkHixcivSCA9pfd_eUp7wkLRhvQ6OtGLAYrWC%3Dk7E76LDQ%40m...
> 
> Is that what people are using?  A missing Author, and co-authors means
> the committer is the author?  Right?  Shouldn't we document this?  That
> does give a unique use for Co-authored-by.

My online checksums commit use a similar pattern, which is how I had
interpreted our use of it.  It lists myself and Magnus as authors with Tomas
Vondra as co-auhor since he provided substantial changes to the patch.

A missing Author tag should IMO always mean that the committer is the author.

>> do think we have enough structured data that if we felt our attribution efforts
>> were insufficient there are more things we could do.  I’m not sure this is the
>> most valuable way to expose this data but it’s a way, we likely don’t do enough
>> promotion even with it, and it seems low maintenance.  But maybe there is a
>> cost/benefit discussion to be had here.
> 
> I guess that is my question.  I don't think the author names have the
> same practical value now that we have commit links, but if people think
> it still has _sufficient_ value, we should keep it --- that was my
> question.

I know from talking to several contributors that seeing their name next to the
feature in the release notes is a huge motivator.

--
Daniel Gustafsson






view thread (53+ messages)  latest in thread

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: [email protected]
  Cc: [email protected], [email protected], [email protected], [email protected], [email protected], [email protected], [email protected]
  Subject: Re: PG 19 release notes and authors
  In-Reply-To: <[email protected]>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox