Fwd: GROUP BY DISTINCT
Pantelis Theodosiou <ypercube@gmail.com>
From: Pantelis Theodosiou <ypercube@gmail.com>
To: pgsql-docs@lists.postgresql.org
Date: 2021-03-13T01:03:19Z
Lists: pgsql-hackers, pgsql-docs
---------- Forwarded message --------- From: Tomas Vondra <tomas.vondra@enterprisedb.com> Date: Fri, Mar 12, 2021 at 11:33 PM Subject: Re: GROUP BY DISTINCT To: Vik Fearing <vik@postgresfriends.org>, Georgios Kokolatos < gkokolatos@protonmail.com>, <pgsql-hackers@lists.postgresql.org> Cc: Erik Rijkers <er@xs4all.nl> Hi Vik, The patch seems quite ready, I have just two comments. 1) Shouldn't this add another <indexterm> for DISTINCT, somewhere in the documentation? Now the index points just to the SELECT DISTINCT part. ..... regards -- Tomas Vondra EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company After reading the above thread in hackers, I noticed that the index does not point to aggrgeate functions either and DISTINCT is not mentioned in the aggregate functions page either: https://www.postgresql.org/docs/current/functions-aggregate.html Shouldn't it be mentioned with an example of COUNT(DISTINCT ...) or aggregate_function(DISTINCT ...) in general ? Best regards Pantelis Theodosiou
Commits
-
Use lfirst_int in cmp_list_len_contents_asc
- b822ae13ea93 14.0 landed
-
Fix misuse of foreach_delete_current().
- 1d581ce7129d 14.0 landed
-
Implement GROUP BY DISTINCT
- be45be9c33a8 14.0 landed