diff options
author | Tom Lane <tgl@sss.pgh.pa.us> | 2016-04-06 11:34:02 -0400 |
---|---|---|
committer | Tom Lane <tgl@sss.pgh.pa.us> | 2016-04-06 11:34:02 -0400 |
commit | de94e2af184e25576b13cbda8cf825118835d1cd (patch) | |
tree | d230afdbb5a4d1f79778107020f4bd2992a4bc71 /src/backend/optimizer/path/joinpath.c | |
parent | d25379eb23383f1d2f969e65e0332b47c19aea94 (diff) | |
download | postgresql-de94e2af184e25576b13cbda8cf825118835d1cd.tar.gz postgresql-de94e2af184e25576b13cbda8cf825118835d1cd.zip |
Run pgindent on a batch of (mostly-planner-related) source files.
Getting annoyed at the amount of unrelated chatter I get from pgindent'ing
Rowley's unique-joins patch. Re-indent all the files it touches.
Diffstat (limited to 'src/backend/optimizer/path/joinpath.c')
-rw-r--r-- | src/backend/optimizer/path/joinpath.c | 95 |
1 files changed, 48 insertions, 47 deletions
diff --git a/src/backend/optimizer/path/joinpath.c b/src/backend/optimizer/path/joinpath.c index f3aced3c931..41b60d01d37 100644 --- a/src/backend/optimizer/path/joinpath.c +++ b/src/backend/optimizer/path/joinpath.c @@ -348,12 +348,12 @@ try_nestloop_path(PlannerInfo *root, */ static void try_partial_nestloop_path(PlannerInfo *root, - RelOptInfo *joinrel, - Path *outer_path, - Path *inner_path, - List *pathkeys, - JoinType jointype, - JoinPathExtraData *extra) + RelOptInfo *joinrel, + Path *outer_path, + Path *inner_path, + List *pathkeys, + JoinType jointype, + JoinPathExtraData *extra) { JoinCostWorkspace workspace; @@ -373,8 +373,8 @@ try_partial_nestloop_path(PlannerInfo *root, } /* - * Before creating a path, get a quick lower bound on what it is likely - * to cost. Bail out right away if it looks terrible. + * Before creating a path, get a quick lower bound on what it is likely to + * cost. Bail out right away if it looks terrible. */ initial_cost_nestloop(root, &workspace, jointype, outer_path, inner_path, @@ -384,17 +384,17 @@ try_partial_nestloop_path(PlannerInfo *root, /* Might be good enough to be worth trying, so let's try it. */ add_partial_path(joinrel, (Path *) - create_nestloop_path(root, - joinrel, - jointype, - &workspace, - extra->sjinfo, - &extra->semifactors, - outer_path, - inner_path, - extra->restrictlist, - pathkeys, - NULL)); + create_nestloop_path(root, + joinrel, + jointype, + &workspace, + extra->sjinfo, + &extra->semifactors, + outer_path, + inner_path, + extra->restrictlist, + pathkeys, + NULL)); } /* @@ -571,8 +571,8 @@ try_partial_hashjoin_path(PlannerInfo *root, } /* - * Before creating a path, get a quick lower bound on what it is likely - * to cost. Bail out right away if it looks terrible. + * Before creating a path, get a quick lower bound on what it is likely to + * cost. Bail out right away if it looks terrible. */ initial_cost_hashjoin(root, &workspace, jointype, hashclauses, outer_path, inner_path, @@ -582,17 +582,17 @@ try_partial_hashjoin_path(PlannerInfo *root, /* Might be good enough to be worth trying, so let's try it. */ add_partial_path(joinrel, (Path *) - create_hashjoin_path(root, - joinrel, - jointype, - &workspace, - extra->sjinfo, - &extra->semifactors, - outer_path, - inner_path, - extra->restrictlist, - NULL, - hashclauses)); + create_hashjoin_path(root, + joinrel, + jointype, + &workspace, + extra->sjinfo, + &extra->semifactors, + outer_path, + inner_path, + extra->restrictlist, + NULL, + hashclauses)); } /* @@ -1189,11 +1189,11 @@ match_unsorted_outer(PlannerInfo *root, } /* - * If the joinrel is parallel-safe and the join type supports nested loops, - * we may be able to consider a partial nestloop plan. However, we can't - * handle JOIN_UNIQUE_OUTER, because the outer path will be partial, and - * therefore we won't be able to properly guarantee uniqueness. Nor can - * we handle extra_lateral_rels, since partial paths must not be + * If the joinrel is parallel-safe and the join type supports nested + * loops, we may be able to consider a partial nestloop plan. However, we + * can't handle JOIN_UNIQUE_OUTER, because the outer path will be partial, + * and therefore we won't be able to properly guarantee uniqueness. Nor + * can we handle extra_lateral_rels, since partial paths must not be * parameterized. */ if (joinrel->consider_parallel && nestjoinOK && @@ -1235,10 +1235,10 @@ consider_parallel_nestloop(PlannerInfo *root, outerpath->pathkeys); /* - * Try the cheapest parameterized paths; only those which will - * produce an unparameterized path when joined to this outerrel - * will survive try_partial_nestloop_path. The cheapest - * unparameterized path is also in this list. + * Try the cheapest parameterized paths; only those which will produce + * an unparameterized path when joined to this outerrel will survive + * try_partial_nestloop_path. The cheapest unparameterized path is + * also in this list. */ foreach(lc2, innerrel->cheapest_parameterized_paths) { @@ -1250,16 +1250,17 @@ consider_parallel_nestloop(PlannerInfo *root, /* * Like match_unsorted_outer, we only consider a single nestloop - * path when the jointype is JOIN_UNIQUE_INNER. But we have to scan - * cheapest_parameterized_paths to find the one we want to consider, - * because cheapest_total_path might not be parallel-safe. + * path when the jointype is JOIN_UNIQUE_INNER. But we have to + * scan cheapest_parameterized_paths to find the one we want to + * consider, because cheapest_total_path might not be + * parallel-safe. */ if (jointype == JOIN_UNIQUE_INNER) { if (!bms_is_empty(PATH_REQ_OUTER(innerpath))) continue; innerpath = (Path *) create_unique_path(root, innerrel, - innerpath, extra->sjinfo); + innerpath, extra->sjinfo); Assert(innerpath); } @@ -1456,8 +1457,8 @@ hash_inner_and_outer(PlannerInfo *root, outerrel->partial_pathlist != NIL && bms_is_empty(joinrel->lateral_relids)) { - Path *cheapest_partial_outer; - Path *cheapest_safe_inner = NULL; + Path *cheapest_partial_outer; + Path *cheapest_safe_inner = NULL; cheapest_partial_outer = (Path *) linitial(outerrel->partial_pathlist); |