Re: v17 Possible Union All Bug
Andrei Lepikhov <a.lepikhov@postgrespro.ru>
From: Andrei Lepikhov <a.lepikhov@postgrespro.ru>
To: Alexander Korotkov <aekorotkov@gmail.com>,
"David G. Johnston" <david.g.johnston@gmail.com>
Cc: David Rowley <dgrowleyml@gmail.com>,
Alexander Korotkov <akorotkov@postgresql.org>,
PostgreSQL Bug List <pgsql-bugs@lists.postgresql.org>
Date: 2024-02-01T04:06:45Z
Lists: pgsql-bugs
Attachments
- unionall-repro-c.sql (application/sql)
On 30/1/2024 04:45, Alexander Korotkov wrote: > On Mon, Jan 29, 2024 at 11:32 PM David G. Johnston > <david.g.johnston@gmail.com> wrote: >> >> On Mon, Jan 29, 2024 at 9:19 AM David G. Johnston <david.g.johnston@gmail.com> wrote: >>> >>> On Fri, Jan 26, 2024 at 5:36 PM David Rowley <dgrowleyml@gmail.com> wrote: >>>> >>>> On Sat, 27 Jan 2024 at 13:19, David Rowley <dgrowleyml@gmail.com> wrote: >>>>> Are the results correct if you SET enable_presorted_aggregate=0;? >>>> >>> >>> Apparently I didn't reply-all... >>> >>> Yes, the problem goes away when I disabled presorted_aggregate >>> >>> I'll see if that knowledge can help build a better reproducer. >>> >> >> I've deferred doing a better reproducer for the moment, I reliably got: >> initdb >> psql --file ~/unionall-repro.sql >> psql -c 'select * from rolegraph.role_graph_broken;' >> >> oid | role_type | rolname | rolsuper | administration >> -------+-----------+--------------------+----------+----------------------------------------------- >> 16390 | User | u6_green_leader_su | f | mog of g6a_fixedops_manager_su from superuser+ >> | | | | mog of g6c_service_manager_su from superuser + >> | | | | mog of g6d_service_advisor_su from superuser + >> | | | | mog of g6e_service_tech_su from superuser + >> | | | | mog of g6c_service_manager_su from cr_admin >> (1 row) >> >> to be produced for the bad bisect result and the correct nested result for *manager* to produce on the good result. > > > Thank you for noticing. I'm investigating this. Very curious bug. I simplified the test a bit (see in attachment), but still can't replace system tables, like pg_authid, with a plain table. Will try further. -- regards, Andrei Lepikhov Postgres Professional
Commits
-
Fix usage of aggregate pathkeys in group_keys_reorder_by_pathkeys()
- c01f6ef46c8f 17.0 landed
-
Move is_valid_ascii() to ascii.h.
- 97287bdfae41 17.0 cited
-
Test EXPLAIN (FORMAT JSON) ... XMLTABLE
- 752533d40fd5 17.0 cited
-
Add test module injection_points
- 49cd2b93d7db 17.0 cited
-
Fix table name collision in tests in 0452b461bc
- c03d91d9be37 17.0 cited
-
Explore alternative orderings of group-by pathkeys during optimization.
- 0452b461bc40 17.0 cited
-
Generalize the common code of adding sort before processing of grouping
- 7ab80ac1caf9 17.0 cited
-
Reorder actions in ProcArrayApplyRecoveryInfo()
- c64086b79dba 17.0 cited
-
More documentation updates for incremental backup.
- 7b1dbf0a8d1d 17.0 cited
-
Avoid useless ReplicationOriginExitCleanup locking
- aa817c749657 17.0 cited