database/postgres
1
0
Fork 0
mirror of https://github.com/postgres/postgres.git synced 2025-08-25 20:23:07 +03:00
Code Activity

Fix Memoize to work with partitionwise joining.

Browse Source 
A couple of places weren't up to speed for this.  By sheer good
luck, we didn't fail but just selected a non-memoized join plan,
at least in the test case we have.  Nonetheless, it's a bug,
and I'm not quite sure that it couldn't have worse consequences
in other examples.  So back-patch to v14 where Memoize came in.

Richard Guo

Discussion: https://postgr.es/m/CAMbWs48GkNom272sfp0-WeD6_0HSR19BJ4H1c9ZKSfbVnJsvRg@mail.gmail.com
This commit is contained in:
Tom Lane
2022-12-05 12:36:40 -05:00
parent 35ce24c333
commit d69d01ba9d
5 changed files with 63 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
src/backend/optimizer/path/joinpath.c
View File

@@ -597,7 +597,8 @@ get_memoize_path(PlannerInfo *root, RelOptInfo *innerrel,
/* Check if we have hash ops for each parameter to the path */
if (paraminfo_get_equal_hashops(root,
inner_path->param_info,
outerrel,
outerrel->top_parent ?
outerrel->top_parent : outerrel,
innerrel,
&param_exprs,
&hash_operators,

1
src/backend/optimizer/util/pathnode.c
View File

@@ -4246,6 +4246,7 @@ do { \
FLAT_COPY_PATH(mpath, path, MemoizePath);
REPARAMETERIZE_CHILD_PATH(mpath->subpath);
ADJUST_CHILD_ATTRS(mpath->param_exprs);
new_path = (Path *) mpath;
}
break;