aboutsummaryrefslogtreecommitdiff
path: root/src/backend/utils/adt/jsonfuncs.c
diff options
context:
space:
mode:
authorTom Lane <tgl@sss.pgh.pa.us>2022-12-04 13:17:18 -0500
committerTom Lane <tgl@sss.pgh.pa.us>2022-12-04 13:17:18 -0500
commitbf8fd64ff5e65f8964d1b52c50019f0198ec5f5e (patch)
tree1294af6863f906a3e960509d455c71cfb025c9c6 /src/backend/utils/adt/jsonfuncs.c
parentc67204db61fa9c4429b30ce031ef833748c8c69a (diff)
downloadpostgresql-bf8fd64ff5e65f8964d1b52c50019f0198ec5f5e.tar.gz
postgresql-bf8fd64ff5e65f8964d1b52c50019f0198ec5f5e.zip
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
Diffstat (limited to 'src/backend/utils/adt/jsonfuncs.c')
0 files changed, 0 insertions, 0 deletions