Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Commit364a9f4

Browse files
committed
Refactor pull_var_clause's API to make it less tedious to extend.
In commit1d97c19 and laterc1d9579, we extendedpull_var_clause's API by adding enum-type arguments. That's sort of a painto maintain, though, because it means every time we add a new behavior wemust touch every last one of the call sites, even if there's a reasonabledefault behavior that most of them could use. Let's switch over to using abitmask of flags, instead; that seems more maintainable and might save ananosecond or two as well. This commit changes no behavior in itself,though I'm going to follow it up with one that does add a new behavior.In passing, remove flatten_tlist(), which has not been used since 9.1and would otherwise need the same API changes.Removing these enums means that optimizer/tlist.h no longer needs todepend on optimizer/var.h. Changing that caused a number of C files toneed addition of #include "optimizer/var.h" (probably we can thank oldruns of pgrminclude for that); but on balance it seems like a good changeanyway.
1 parent37c5486 commit364a9f4

File tree

20 files changed

+94
-128
lines changed

20 files changed

+94
-128
lines changed

‎contrib/postgres_fdw/deparse.c‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -734,7 +734,6 @@ build_tlist_to_deparse(RelOptInfo *foreignrel)
734734
tlist=add_to_flat_tlist(tlist,foreignrel->reltarget.exprs);
735735
tlist=add_to_flat_tlist(tlist,
736736
pull_var_clause((Node*)fpinfo->local_conds,
737-
PVC_REJECT_AGGREGATES,
738737
PVC_RECURSE_PLACEHOLDERS));
739738

740739
returntlist;

‎src/backend/catalog/heap.c‎

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -2006,9 +2006,7 @@ StoreRelCheck(Relation rel, char *ccname, Node *expr,
20062006
* in check constraints; it would fail to examine the contents of
20072007
* subselects.
20082008
*/
2009-
varList=pull_var_clause(expr,
2010-
PVC_REJECT_AGGREGATES,
2011-
PVC_REJECT_PLACEHOLDERS);
2009+
varList=pull_var_clause(expr,0);
20122010
keycount=list_length(varList);
20132011

20142012
if (keycount>0)
@@ -2323,9 +2321,7 @@ AddRelationNewConstraints(Relation rel,
23232321
List*vars;
23242322
char*colname;
23252323

2326-
vars=pull_var_clause(expr,
2327-
PVC_REJECT_AGGREGATES,
2328-
PVC_REJECT_PLACEHOLDERS);
2324+
vars=pull_var_clause(expr,0);
23292325

23302326
/* eliminate duplicates */
23312327
vars=list_union(NIL,vars);

‎src/backend/commands/trigger.c‎

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -352,9 +352,7 @@ CreateTrigger(CreateTrigStmt *stmt, const char *queryString,
352352
* subselects in WHEN clauses; it would fail to examine the contents
353353
* of subselects.
354354
*/
355-
varList=pull_var_clause(whenClause,
356-
PVC_REJECT_AGGREGATES,
357-
PVC_REJECT_PLACEHOLDERS);
355+
varList=pull_var_clause(whenClause,0);
358356
foreach(lc,varList)
359357
{
360358
Var*var= (Var*)lfirst(lc);

‎src/backend/optimizer/path/allpaths.c‎

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2509,9 +2509,7 @@ qual_is_pushdown_safe(Query *subquery, Index rti, Node *qual,
25092509
* Examine all Vars used in clause; since it's a restriction clause, all
25102510
* such Vars must refer to subselect output columns.
25112511
*/
2512-
vars=pull_var_clause(qual,
2513-
PVC_REJECT_AGGREGATES,
2514-
PVC_INCLUDE_PLACEHOLDERS);
2512+
vars=pull_var_clause(qual,PVC_INCLUDE_PLACEHOLDERS);
25152513
foreach(vl,vars)
25162514
{
25172515
Var*var= (Var*)lfirst(vl);

‎src/backend/optimizer/path/equivclass.c‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -910,7 +910,7 @@ generate_base_implied_equalities_no_const(PlannerInfo *root,
910910
{
911911
EquivalenceMember*cur_em= (EquivalenceMember*)lfirst(lc);
912912
List*vars=pull_var_clause((Node*)cur_em->em_expr,
913-
PVC_RECURSE_AGGREGATES,
913+
PVC_RECURSE_AGGREGATES |
914914
PVC_INCLUDE_PLACEHOLDERS);
915915

916916
add_vars_to_targetlist(root,vars,ec->ec_relids, false);

‎src/backend/optimizer/plan/analyzejoins.c‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,7 @@
2929
#include"optimizer/paths.h"
3030
#include"optimizer/planmain.h"
3131
#include"optimizer/tlist.h"
32+
#include"optimizer/var.h"
3233
#include"utils/lsyscache.h"
3334

3435
/* local functions */

‎src/backend/optimizer/plan/createplan.c‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5335,7 +5335,7 @@ prepare_sort_from_pathkeys(Plan *lefttree, List *pathkeys,
53355335

53365336
sortexpr=em->em_expr;
53375337
exprvars=pull_var_clause((Node*)sortexpr,
5338-
PVC_INCLUDE_AGGREGATES,
5338+
PVC_INCLUDE_AGGREGATES |
53395339
PVC_INCLUDE_PLACEHOLDERS);
53405340
foreach(k,exprvars)
53415341
{

‎src/backend/optimizer/plan/initsplan.c‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -146,7 +146,7 @@ void
146146
build_base_rel_tlists(PlannerInfo*root,List*final_tlist)
147147
{
148148
List*tlist_vars=pull_var_clause((Node*)final_tlist,
149-
PVC_RECURSE_AGGREGATES,
149+
PVC_RECURSE_AGGREGATES |
150150
PVC_INCLUDE_PLACEHOLDERS);
151151

152152
if (tlist_vars!=NIL)
@@ -161,7 +161,7 @@ build_base_rel_tlists(PlannerInfo *root, List *final_tlist)
161161
if (root->parse->havingQual)
162162
{
163163
List*having_vars=pull_var_clause(root->parse->havingQual,
164-
PVC_RECURSE_AGGREGATES,
164+
PVC_RECURSE_AGGREGATES |
165165
PVC_INCLUDE_PLACEHOLDERS);
166166

167167
if (having_vars!=NIL)
@@ -1787,7 +1787,7 @@ distribute_qual_to_rels(PlannerInfo *root, Node *clause,
17871787
if (bms_membership(relids)==BMS_MULTIPLE)
17881788
{
17891789
List*vars=pull_var_clause(clause,
1790-
PVC_RECURSE_AGGREGATES,
1790+
PVC_RECURSE_AGGREGATES |
17911791
PVC_INCLUDE_PLACEHOLDERS);
17921792

17931793
add_vars_to_targetlist(root,vars,relids, false);

‎src/backend/optimizer/plan/planner.c‎

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -43,6 +43,7 @@
4343
#include"optimizer/prep.h"
4444
#include"optimizer/subselect.h"
4545
#include"optimizer/tlist.h"
46+
#include"optimizer/var.h"
4647
#include"parser/analyze.h"
4748
#include"parser/parsetree.h"
4849
#include"parser/parse_agg.h"
@@ -3840,7 +3841,7 @@ make_group_input_target(PlannerInfo *root, List *tlist)
38403841
* pulled out here, too.
38413842
*/
38423843
non_group_vars=pull_var_clause((Node*)non_group_cols,
3843-
PVC_RECURSE_AGGREGATES,
3844+
PVC_RECURSE_AGGREGATES |
38443845
PVC_INCLUDE_PLACEHOLDERS);
38453846
sub_tlist=add_to_flat_tlist(sub_tlist,non_group_vars);
38463847

@@ -4088,7 +4089,7 @@ make_window_input_target(PlannerInfo *root,
40884089
* at higher levels.
40894090
*/
40904091
flattenable_vars=pull_var_clause((Node*)flattenable_cols,
4091-
PVC_INCLUDE_AGGREGATES,
4092+
PVC_INCLUDE_AGGREGATES |
40924093
PVC_INCLUDE_PLACEHOLDERS);
40934094
new_tlist=add_to_flat_tlist(new_tlist,flattenable_vars);
40944095

‎src/backend/optimizer/plan/setrefs.c‎

Lines changed: 8 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1641,12 +1641,12 @@ set_join_references(PlannerInfo *root, Join *join, int rtoffset)
16411641
*
16421642
* In most cases, we have to match up individual Vars in the tlist and
16431643
* qual expressions with elements of the subplan's tlist (which was
1644-
* generated byflatten_tlist() fromthese selfsame expressions, so it
1645-
*should have allthe required variables). There is an important exception,
1646-
*however: GROUP BY and ORDER BY expressions will have been pushed into the
1647-
* subplan tlist unflattened. If these values are also needed in the output
1648-
* then we want to reference the subplan tlist element rather than recomputing
1649-
* the expression.
1644+
* generated byflatteningthese selfsame expressions, so it should have all
1645+
* the required variables). There is an important exception, however:
1646+
*depending on where we are in the plan tree, sort/group columns may have
1647+
*been pushed into thesubplan tlist unflattened. If these values are also
1648+
*needed in the outputthen we want to reference the subplan tlist element
1649+
*rather than recomputingthe expression.
16501650
*/
16511651
staticvoid
16521652
set_upper_references(PlannerInfo*root,Plan*plan,intrtoffset)
@@ -2129,7 +2129,8 @@ fix_join_expr_mutator(Node *node, fix_join_expr_context *context)
21292129
*
21302130
* An error is raised if no matching var can be found in the subplan tlist
21312131
* --- so this routine should only be applied to nodes whose subplans'
2132-
* targetlists were generated via flatten_tlist() or some such method.
2132+
* targetlists were generated by flattening the expressions used in the
2133+
* parent node.
21332134
*
21342135
* If itlist->has_non_vars is true, then we try to match whole subexpressions
21352136
* against elements of the subplan tlist, so that we can avoid recomputing

0 commit comments

Comments
 (0)
[8]ページ先頭
©2009-2025 Movatter.jp