Fix generate_partitionwise_join_paths() to tolerate failure.
authorTom Lane <tgl@sss.pgh.pa.us>
Sun, 4 Dec 2022 18:17:18 +0000 (13:17 -0500)
committerTom Lane <tgl@sss.pgh.pa.us>
Sun, 4 Dec 2022 18:17:18 +0000 (13:17 -0500)
commitbf8fd64ff5e65f8964d1b52c50019f0198ec5f5e
tree1294af6863f906a3e960509d455c71cfb025c9c6
parentc67204db61fa9c4429b30ce031ef833748c8c69a
Fix generate_partitionwise_join_paths() to tolerate failure.

We might fail to generate a partitionwise join, because
reparameterize_path_by_child() does not support all path types.
This should not be a hard failure condition: we should just fall back
to a non-partitioned join.  However, generate_partitionwise_join_paths
did not consider this possibility and would emit the (misleading)
error "could not devise a query plan for the given query" if we'd
failed to make any paths for a child join.  Fix it to give up on
partitionwise joining if so.  (The accepted technique for giving up
appears to be to set rel->nparts = 0, which I find pretty bizarre,
but there you have it.)

I have not added a test case because there'd be little point:
any omissions of this sort that we identify would soon get fixed
by extending reparameterize_path_by_child(), so the test would stop
proving anything.  However, right now there is a known test case based
on failure to cover MaterialPath, and with that I've found that this
is broken in all supported versions.  Hence, patch all the way back.

Original report and patch by me; thanks to Richard Guo for
identifying a test case that works against committed versions.

Discussion: https://postgr.es/m/1854233.1669949723@sss.pgh.pa.us
src/backend/optimizer/path/allpaths.c