public inbox for [email protected]  

help / color / mirror / Atom feed

From: Peter Eisentraut <[email protected]>
To: Greg Sabino Mullane <[email protected]>
Cc: [email protected] <[email protected]>
Cc: Tom Lane <[email protected]>
Subject: Re: [PATCH] Provide support for trailing commas
Date: Fri, 27 Mar 2026 17:08:41 +0100
Message-ID: <[email protected]> (raw)
In-Reply-To: <CAKAnmmKWEffLVRRSYkqKMKoK6RoZc1ecT_rPzMkVsuqOgzbjog@mail.gmail.com>
References: <CAKAnmmKVwP=yr8t8p8_QtrBFjyENLZyPxscJh4UrkCeqMQuhrw@mail.gmail.com>
	<[email protected]>
	<[email protected]>
	<CAKAnmmK+1Rxv3FSKCku=ePNcwkcd6fqZ5C=y7_3wjyd4+BbwhQ@mail.gmail.com>
	<[email protected]>
	<CAKAnmmKWEffLVRRSYkqKMKoK6RoZc1ecT_rPzMkVsuqOgzbjog@mail.gmail.com>

On 23.03.26 23:54, Greg Sabino Mullane wrote:
> Alright, I walked through the entire list of every place the original 
> patch added a comma, and ruthlessly trimmed out the ones that were not 
> really needed, likely to cause issues, and/or frankly applied on too 
> trivial a piece of SQL to worry about. Here's my list that people can 
> poke at. Once we settle on which ones survive, I can make a new patch. 
> The focus is mostly on the largest potential uses: lists of select 
> items / column names.

I wrote:

 > I think there is consensus for allowing a trailing comma in the 
SELECT > list.  This would be the target_list production in gram.y.

I think that's the only one there is consensus about.

(But target_list is also used in RETURNING, so maybe the productions 
would need to be split up.)

view thread (17+ messages)

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