public inbox for [email protected]  

help / color / mirror / Atom feed

From: Pavel Stehule <[email protected]>
To: Tom Lane <[email protected]>
Cc: Greg Sabino Mullane <[email protected]>
Cc: [email protected] <[email protected]>
Subject: Re: [PATCH] Provide support for trailing commas
Date: Wed, 7 Jan 2026 05:58:32 +0100
Message-ID: <CAFj8pRDN7dtu3LdMtgmsayJkOKyOOT57oh=JQ-J_rH69V5e6TQ@mail.gmail.com> (raw)
In-Reply-To: <[email protected]>
References: <CAKAnmmKVwP=yr8t8p8_QtrBFjyENLZyPxscJh4UrkCeqMQuhrw@mail.gmail.com>
	<[email protected]>

út 6. 1. 2026 v 21:49 odesílatel Tom Lane <[email protected]> napsal:

> Greg Sabino Mullane <[email protected]> writes:
> > 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:
>
> > SELECT 1,2,3, FROM pg_class, ORDER BY greatest(relpages,1,);
>
> Did we not discuss and reject this very idea in the past?
> A quick search finds [1][2], and there may be more threads.
>
> > This is not as trivial a task as it seems, as those who have dabbled with
> > our parser (or parsers in general) may suspect.
>
> Indeed.  I think it carries substantial risk of creating large parsing
> problems down the road, when we try to integrate whatever random
> syntax the SQL committee comes up with next.  And frankly, I don't
> agree that it'd be beneficial.
>

+1

Pavel


>
>                         regards, tom lane
>
> [1]
> https://www.postgresql.org/message-id/flat/20140928114246.GA22150%40artax.karlin.mff.cuni.cz
> [2]
> https://www.postgresql.org/message-id/flat/0368c60f-abe2-4f5f-972d-7cd1e6db2382%40gmail.com
>
>
>

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: <CAFj8pRDN7dtu3LdMtgmsayJkOKyOOT57oh=JQ-J_rH69V5e6TQ@mail.gmail.com>

* 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