Re: GROUP BY DISTINCT
Thomas Munro <thomas.munro@gmail.com>
From: Thomas Munro <thomas.munro@gmail.com>
To: Tomas Vondra <tomas.vondra@enterprisedb.com>
Cc: Vik Fearing <vik@postgresfriends.org>,
Georgios Kokolatos <gkokolatos@protonmail.com>, PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>,
Erik Rijkers <er@xs4all.nl>
Date: 2021-03-18T21:02:53Z
Lists: pgsql-hackers, pgsql-docs
On Fri, Mar 19, 2021 at 8:27 AM Tomas Vondra <tomas.vondra@enterprisedb.com> wrote: > Hmmm, this seems to fail on lapwing with this error: > > parse_agg.c: In function 'expand_grouping_sets': > parse_agg.c:1851:23: error: value computed is not used > [-Werror=unused-value] > cc1: all warnings being treated as errors > > That line is this: > > foreach_delete_current(result, cell); > > and I don't see how any of the values close by could be unused ... > > The only possibility I can think of is some sort of issue in the old-ish > gcc release (4.7.2). No sure what's going on there, but data points: I tried a 32 bit build here (that's the other special thing about lapwing) and didn't see the warning. GCC 10. Also curculio (gcc 4.2) and snapper (gcc 4.7) are also showing this warning, but they don't have -Werror so they don't fail. sidewinder (gcc 4.8) is not showing the warning.
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