public inbox for [email protected]  

help / color / mirror / Atom feed

From: Andres Freund <[email protected]>
To: [email protected]
To: Bruce Momjian <[email protected]>
To: Greg Sabino Mullane <[email protected]>
Cc: [email protected] <[email protected]>
Subject: Re: [PATCH] Provide support for trailing commas
Date: Thu, 08 Jan 2026 00:51:14 -0500
Message-ID: <[email protected]> (raw)
In-Reply-To: <[email protected]>
References: <CAKAnmmKVwP=yr8t8p8_QtrBFjyENLZyPxscJh4UrkCeqMQuhrw@mail.gmail.com>
	<[email protected]>

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.

Greetings, 

Andres
-- 
Sent from my Android device with K-9 Mail. Please excuse my brevity.

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