Horton

Re: GROUP BY DISTINCT

Erik Rijkers <er@xs4all.nl>

View thread View on postgr.es
From: er@xs4all.nl
To: Vik Fearing <vik@postgresfriends.org>, PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Date: 2021-02-21T14:06:03Z
Lists: pgsql-hackers, pgsql-docs 
> On 2021.02.21. 13:52 Vik Fearing <vik@postgresfriends.org> wrote:
>  
> Attached is a patch to implement this for PostgreSQL.
> []

The changed line that gets stuffed into sql_features is missing a terminal value (to fill the 'comments' column).
This line:
'+T434	GROUP BY DISTINCT			YES'

(A tab at the end will do, I suppose; that's how I fixed the patch locally)

Erik Rijkers

Commits

  1. Use lfirst_int in cmp_list_len_contents_asc

  2. Fix misuse of foreach_delete_current().

  3. Implement GROUP BY DISTINCT