diff options
| author | Richard Guo <rguo@postgresql.org> | 2025-05-08 18:21:32 +0900 |
|---|---|---|
| committer | Richard Guo <rguo@postgresql.org> | 2025-05-08 18:21:32 +0900 |
| commit | c06e909c26f070dee78f73c35565d6f4a4ffdcda (patch) | |
| tree | 9f69470d8ba03027f37796542b8c0e7bafb2f44e /src/backend/foreign/foreign.c | |
| parent | 773db22269d474fab46d25e9e15b1e55252cf92c (diff) | |
| download | postgresql-c06e909c26f070dee78f73c35565d6f4a4ffdcda.tar.gz postgresql-c06e909c26f070dee78f73c35565d6f4a4ffdcda.zip | |
Track the number of presorted outer pathkeys in MergePath
When creating an explicit Sort node for the outer path of a mergejoin,
we need to determine the number of presorted keys of the outer path to
decide whether explicit incremental sort can be applied. Currently,
this is done by repeatedly calling pathkeys_count_contained_in.
This patch caches the number of presorted outer pathkeys in MergePath,
allowing us to save several calls to pathkeys_count_contained_in. It
can be considered a complement to the changes in commit 828e94c9d.
Reported-by: David Rowley <dgrowleyml@gmail.com>
Author: Richard Guo <guofenglinux@gmail.com>
Reviewed-by: Tender Wang <tndrwang@gmail.com>
Discussion: https://postgr.es/m/CAApHDvqvBireB_w6x8BN5txdvBEHxVgZBt=rUnpf5ww5P_E_ww@mail.gmail.com
Diffstat (limited to 'src/backend/foreign/foreign.c')
| -rw-r--r-- | src/backend/foreign/foreign.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/src/backend/foreign/foreign.c b/src/backend/foreign/foreign.c index 9c57f7a259c..a57e59f27ea 100644 --- a/src/backend/foreign/foreign.c +++ b/src/backend/foreign/foreign.c @@ -821,8 +821,9 @@ GetExistingLocalJoinPath(RelOptInfo *joinrel) * for the mergejoin, we can skip doing an explicit sort. */ if (merge_path->outersortkeys && - pathkeys_contained_in(merge_path->outersortkeys, - joinpath->outerjoinpath->pathkeys)) + pathkeys_count_contained_in(merge_path->outersortkeys, + joinpath->outerjoinpath->pathkeys, + &merge_path->outer_presorted_keys)) merge_path->outersortkeys = NIL; } } |