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.)
