public inbox for [email protected]  

help / color / mirror / Atom feed

From: Jelte Fennema-Nio <[email protected]>
To: Bruce Momjian <[email protected]>
Cc: Andres Freund <[email protected]>
Cc: [email protected]
Cc: Greg Sabino Mullane <[email protected]>
Subject: Re: [PATCH] Provide support for trailing commas
Date: Thu, 8 Jan 2026 16:40:35 +0100
Message-ID: <CAGECzQSq879_RDW1DTaTJs9D9n94h4kS7UrAU8e=nS4g4jf9_A@mail.gmail.com> (raw)
In-Reply-To: <[email protected]>
References: <CAKAnmmKVwP=yr8t8p8_QtrBFjyENLZyPxscJh4UrkCeqMQuhrw@mail.gmail.com>
	<[email protected]>
	<[email protected]>
	<[email protected]>

On Thu, 8 Jan 2026 at 16:03, Bruce Momjian <[email protected]> wrote:
> I disagree.  It was clear in this case that it was the concept that was
> not wanted, rather than the implementation cost.

It definitely wasn't clear IMO. See the quote from Tom, that Greg
shared from that thread. Tom at that point said he'd be in favor, IF
it would be done everywhere. Which is exactly what Greg did.

Ofcourse Tom is allowed to change his opinion, but if I'd felt
strongly enough about this then I'd also have implemented this first
based on  the previous thread.

For the record, I'd very much like trailing commas to work. Just not
enough to put the time in to discuss them at length. But I'd
definitely like to not get errors thrown in my face every time I do a
trivial modification of a query, and I'd also like my git diffs not to
touch lines that logically have nothing to do with the change I'm
actually doing.

view thread (17+ 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]
  Subject: Re: [PATCH] Provide support for trailing commas
  In-Reply-To: <CAGECzQSq879_RDW1DTaTJs9D9n94h4kS7UrAU8e=nS4g4jf9_A@mail.gmail.com>

* 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