Feedback-ID: ie0a040ee:Fastmail
Message-ID: <ab27a660-72b4-41b6-9279-2f007e065fed@eisentraut.org>
Date: Tue, 17 Feb 2026 09:30:02 +0100
MIME-Version: 1.0
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH] Provide support for trailing commas
To: Greg Sabino Mullane <htamfids@gmail.com>
Cc: "pgsql-hackers@lists.postgresql.org"
 <pgsql-hackers@lists.postgresql.org>, Tom Lane <tgl@sss.pgh.pa.us>
References: <CAKAnmmKVwP=yr8t8p8_QtrBFjyENLZyPxscJh4UrkCeqMQuhrw@mail.gmail.com>
 <4042a046-3dc9-4fc6-bfc6-62beb8492f27@eisentraut.org>
 <2228785.1769016115@sss.pgh.pa.us>
 <CAKAnmmK+1Rxv3FSKCku=ePNcwkcd6fqZ5C=y7_3wjyd4+BbwhQ@mail.gmail.com>
Content-Language: en-US
From: Peter Eisentraut <peter@eisentraut.org>
In-Reply-To: <CAKAnmmK+1Rxv3FSKCku=ePNcwkcd6fqZ5C=y7_3wjyd4+BbwhQ@mail.gmail.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Archived-At: <https://www.postgresql.org/message-id/ab27a660-72b4-41b6-9279-2f007e065fed%40eisentraut.org>
Precedence: bulk

On 03.02.26 18:28, Greg Sabino Mullane wrote:
> Having reread this thread, I'm not sure where we are at the moment. I'm 
> happy to support a subset of things for the first run, and I think that 
> will cover 99% of the use cases that people tend to complain about. So a 
> small hit for consistency, but an added bonus of a little protection 
> should the committee do something crazy.

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

Looking into your patch, it looks pretty much like how I would have 
implemented it, so if you could cut down your patch to just this 
production, and add a small amount of documentation and tests, then I 
think we could move forward with it.