]> granicus.if.org Git - postgresql/commit
Prevent planagg.c from failing on queries containing CTEs.
authorTom Lane <tgl@sss.pgh.pa.us>
Tue, 13 Dec 2016 18:20:16 +0000 (13:20 -0500)
committerTom Lane <tgl@sss.pgh.pa.us>
Tue, 13 Dec 2016 18:20:37 +0000 (13:20 -0500)
commit1f542a2eacca030c676cbb594f3b362d43f2f857
tree57830258483cd4d8c849448aae4dab577621236a
parent501c7b94bcb00cfa0faad60135cf6af82fd56a3a
Prevent planagg.c from failing on queries containing CTEs.

The existing tests in preprocess_minmax_aggregates() usually prevent it
from trying to do anything with queries containing CTEs, but there's an
exception: a CTE could be present as a member of an appendrel, if we
flattened a UNION ALL that contains CTE references.  If it did try to
generate an optimized path for a query using a CTE, it failed with
"could not find plan for CTE", as reported by Torsten Förtsch.

The proximate cause is an unwise decision in commit 3fc6e2d7f to clear
subroot->cte_plan_ids in build_minmax_path().  That left the subroot's
cte_plan_ids list out of step with its parse->cteList.

Removing the "subroot->cte_plan_ids = NIL;" assignment is enough to let
the case work again, but really it's pretty silly to be expending any
cycles at all in this module when there are CTEs: we always treat their
outputs as unordered so there's no way for the optimization to win.
Hence, also add an early-exit test so we don't waste time like that.

Back-patch to 9.6 where the misbehavior was introduced.

Report: https://postgr.es/m/CAKkG4_=gjY5QiHtqSZyWMwDuTd_CftKoTaCqxjJ7uUz1-Gw=qw@mail.gmail.com
src/backend/optimizer/plan/planagg.c