public inbox for [email protected]  

help / color / mirror / Atom feed

From: Bruce Momjian <[email protected]>
To: 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 10:03:22 -0500
Message-ID: <[email protected]> (raw)
In-Reply-To: <[email protected]>
References: <CAKAnmmKVwP=yr8t8p8_QtrBFjyENLZyPxscJh4UrkCeqMQuhrw@mail.gmail.com>
	<[email protected]>
	<[email protected]>

On Thu, Jan  8, 2026 at 12:51:14AM -0500, Andres Freund wrote:
> Hi, 
> 
> On January 7, 2026 11:44:29 PM EST, Bruce Momjian <[email protected]> wrote:
> >On Tue, Jan  6, 2026 at 03:35:38PM -0500, Greg Sabino Mullane wrote:
> >> tl;dr Provide support for trailing commas, where possible and practical.
> >> 
> >> (Disclaimer: No LLM or AI was used to craft this patch or this email)
> >> 
> >> Happy New Year! Please find attached a patch to provide comprehensive trailing
> >> comma support, where we allow (when possible) a comma that does NOT come before
> >> the next item in a list. To wit:
> >
> >I am unclear why you bothered to create and work on such a large email
> >and patch rather than ask if this was a desired change;  see:
> >
> >	https://wiki.postgresql.org/wiki/Todo#Development_Process
> 
> Meh. This kind of change really needs information about the
> implementation cost to be judged. The process described there rarely is
> a good idea for anybody that has some community experience.

I disagree.  It was clear in this case that it was the concept that was
not wanted, rather than the implementation cost.

-- 
  Bruce Momjian  <[email protected]>        https://momjian.us
  EDB                                      https://enterprisedb.com

  Do not let urgent matters crowd out time for investment in the future.

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]
  Subject: Re: [PATCH] Provide support for trailing commas
  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