pg.ddx.io  pgsql-docs@postgresql.org mailing list archive  
help / color / mirror / Atom feed
From: Vik Fearing <vik@postgresfriends.org>
To: er@xs4all.nl, PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Subject: Re: GROUP BY DISTINCT
Date: Sun, 21 Feb 2021 15:14:12 +0100
Message-ID: <c350ff6b-d70d-9952-be23-a18855b62589@postgresfriends.org> (raw)
In-Reply-To: <2056725465.229405.1613916363889@webmailclassic.xs4all.nl>
References: <bf3805a8-d7d1-ae61-fece-761b7ff41ecc@postgresfriends.org>
	<2056725465.229405.1613916363889@webmailclassic.xs4all.nl>

On 2/21/21 3:06 PM, er@xs4all.nl wrote:
>> 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)

Argh.  Fixed.

Thank you for looking at it!
-- 
Vik Fearing

Attachments:

  [text/x-patch] 0001-implement-GROUP-BY-DISTINCT.v02.patch (0B, ../c350ff6b-d70d-9952-be23-a18855b62589@postgresfriends.org/2-0001-implement-GROUP-BY-DISTINCT.v02.patch)
  download

view thread (20+ messages)  latest in thread

Message-ID: <c350ff6b-d70d-9952-be23-a18855b62589@postgresfriends.org>
Permalink:  ../c350ff6b-d70d-9952-be23-a18855b62589@postgresfriends.org/
Also on:    postgresql.org/message-id/c350ff6b-d70d-9952-be23-a18855b62589@postgresfriends.org

 ·  · 

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: pgsql-docs@postgresql.org
  Cc: vik@postgresfriends.org, pgsql-hackers@lists.postgresql.org
  Subject: Re: GROUP BY DISTINCT
  In-Reply-To: <c350ff6b-d70d-9952-be23-a18855b62589@postgresfriends.org>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox