Re: [PATCH] Provide support for trailing commas - Mailing list pgsql-hackers

From Peter Eisentraut
Subject Re: [PATCH] Provide support for trailing commas
Date
Msg-id ab27a660-72b4-41b6-9279-2f007e065fed@eisentraut.org
Whole thread Raw
In response to Re: [PATCH] Provide support for trailing commas  (Greg Sabino Mullane <htamfids@gmail.com>)
Responses Re: [PATCH] Provide support for trailing commas
List pgsql-hackers
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.




pgsql-hackers by date:

Previous
From: Andreas Karlsson
Date:
Subject: Re: Our ABI diff infrastructure ignores enum SysCacheIdentifier
Next
From: Pavlo Golub
Date:
Subject: Re: [PATCH] Add last_executed timestamp to pg_stat_statements