diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2021-04-20 11:37:36 -0400 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2021-04-20 11:37:36 -0400 |
| commit | 7645376774c8532159f5f0f905e5e734d4ccbb18 (patch) | |
| tree | c68685171724f4331f3dec33777c36af86237822 /src/include/optimizer/paths.h | |
| parent | 375398244168add84a884347625d14581a421e71 (diff) | |
| download | postgresql-7645376774c8532159f5f0f905e5e734d4ccbb18.tar.gz postgresql-7645376774c8532159f5f0f905e5e734d4ccbb18.zip | |
Rename find_em_expr_usable_for_sorting_rel.
I didn't particularly like this function name, as it fails to
express what's going on. Also, returning the sort expression
alone isn't too helpful --- typically, a caller would also
need some other fields of the EquivalenceMember. But the
sole caller really only needs a bool result, so let's make
it "bool relation_can_be_sorted_early()".
Discussion: https://postgr.es/m/91f3ec99-85a4-fa55-ea74-33f85a5c651f@swarm64.com
Diffstat (limited to 'src/include/optimizer/paths.h')
| -rw-r--r-- | src/include/optimizer/paths.h | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/src/include/optimizer/paths.h b/src/include/optimizer/paths.h index 888e85ff5b3..f1d111063c2 100644 --- a/src/include/optimizer/paths.h +++ b/src/include/optimizer/paths.h @@ -144,10 +144,9 @@ extern EquivalenceMember *find_computable_ec_member(PlannerInfo *root, Relids relids, bool require_parallel_safe); extern Expr *find_em_expr_for_rel(EquivalenceClass *ec, RelOptInfo *rel); -extern Expr *find_em_expr_usable_for_sorting_rel(PlannerInfo *root, - EquivalenceClass *ec, - RelOptInfo *rel, - bool require_parallel_safe); +extern bool relation_can_be_sorted_early(PlannerInfo *root, RelOptInfo *rel, + EquivalenceClass *ec, + bool require_parallel_safe); extern void generate_base_implied_equalities(PlannerInfo *root); extern List *generate_join_implied_equalities(PlannerInfo *root, Relids join_relids, |