From 18fbe4142faf32228ad40881eb61bb35bf523e61 Mon Sep 17 00:00:00 2001 From: Bruce Momjian Date: Thu, 4 Feb 1999 01:47:02 +0000 Subject: [PATCH] More optimizer renaming HInfo -> HashInfo. --- src/backend/nodes/copyfuncs.c | 14 +++++++------- src/backend/nodes/equalfuncs.c | 12 ++++++------ src/backend/nodes/outfuncs.c | 10 +++++----- src/backend/nodes/readfuncs.c | 16 ++++++++-------- src/backend/optimizer/README | 11 ++++++----- src/backend/optimizer/path/hashutils.c | 16 ++++++++-------- src/backend/optimizer/path/joinpath.c | 6 +++--- src/backend/optimizer/path/prune.c | 9 ++++----- src/backend/optimizer/util/pathnode.c | 6 +++--- src/include/nodes/nodes.h | 4 ++-- src/include/nodes/relation.h | 18 +++++++++--------- 11 files changed, 61 insertions(+), 61 deletions(-) diff --git a/src/backend/nodes/copyfuncs.c b/src/backend/nodes/copyfuncs.c index 49c7b05b3d..7935d6b995 100644 --- a/src/backend/nodes/copyfuncs.c +++ b/src/backend/nodes/copyfuncs.c @@ -7,7 +7,7 @@ * * * IDENTIFICATION - * $Header: /cvsroot/pgsql/src/backend/nodes/copyfuncs.c,v 1.59 1999/02/03 21:16:17 momjian Exp $ + * $Header: /cvsroot/pgsql/src/backend/nodes/copyfuncs.c,v 1.60 1999/02/04 01:46:53 momjian Exp $ * *------------------------------------------------------------------------- */ @@ -1378,13 +1378,13 @@ _copyJoinMethod(JoinMethod *from) } /* ---------------- - * _copyHInfo + * _copyHashInfo * ---------------- */ -static HInfo * -_copyHInfo(HInfo *from) +static HashInfo * +_copyHashInfo(HashInfo *from) { - HInfo *newnode = makeNode(HInfo); + HashInfo *newnode = makeNode(HashInfo); /* ---------------- * copy remainder of node @@ -1797,8 +1797,8 @@ copyObject(void *from) case T_JoinMethod: retval = _copyJoinMethod(from); break; - case T_HInfo: - retval = _copyHInfo(from); + case T_HashInfo: + retval = _copyHashInfo(from); break; case T_MInfo: retval = _copyMInfo(from); diff --git a/src/backend/nodes/equalfuncs.c b/src/backend/nodes/equalfuncs.c index b8727f7f33..116d19327d 100644 --- a/src/backend/nodes/equalfuncs.c +++ b/src/backend/nodes/equalfuncs.c @@ -7,7 +7,7 @@ * * * IDENTIFICATION - * $Header: /cvsroot/pgsql/src/backend/nodes/equalfuncs.c,v 1.22 1999/02/03 20:15:22 momjian Exp $ + * $Header: /cvsroot/pgsql/src/backend/nodes/equalfuncs.c,v 1.23 1999/02/04 01:46:54 momjian Exp $ * *------------------------------------------------------------------------- */ @@ -476,10 +476,10 @@ _equalMergeOrder(MergeOrder *a, MergeOrder *b) } static bool -_equalHInfo(HInfo *a, HInfo *b) +_equalHashInfo(HashInfo *a, HashInfo *b) { - Assert(IsA(a, HInfo)); - Assert(IsA(b, HInfo)); + Assert(IsA(a, HashInfo)); + Assert(IsA(b, HashInfo)); if (a->hashop != b->hashop) return false; @@ -696,8 +696,8 @@ equal(void *a, void *b) case T_MergeOrder: retval = _equalMergeOrder(a, b); break; - case T_HInfo: - retval = _equalHInfo(a, b); + case T_HashInfo: + retval = _equalHashInfo(a, b); break; case T_IndexScan: retval = _equalIndexScan(a, b); diff --git a/src/backend/nodes/outfuncs.c b/src/backend/nodes/outfuncs.c index 7c370053c1..8c0d49a7c3 100644 --- a/src/backend/nodes/outfuncs.c +++ b/src/backend/nodes/outfuncs.c @@ -5,7 +5,7 @@ * * Copyright (c) 1994, Regents of the University of California * - * $Id: outfuncs.c,v 1.63 1999/02/03 20:15:22 momjian Exp $ + * $Id: outfuncs.c,v 1.64 1999/02/04 01:46:54 momjian Exp $ * * NOTES * Every (plan) node in POSTGRES has an associated "out" routine which @@ -1137,10 +1137,10 @@ _outJoinMethod(StringInfo str, JoinMethod *node) } /* - * HInfo is a subclass of JoinMethod. + * HashInfo is a subclass of JoinMethod. */ static void -_outHInfo(StringInfo str, HInfo *node) +_outHashInfo(StringInfo str, HashInfo *node) { appendStringInfo(str, " HASHINFO :hashop %u :jmkeys ", node->hashop); _outNode(str, node->jmethod.jmkeys); @@ -1578,8 +1578,8 @@ _outNode(StringInfo str, void *obj) case T_JoinMethod: _outJoinMethod(str, obj); break; - case T_HInfo: - _outHInfo(str, obj); + case T_HashInfo: + _outHashInfo(str, obj); break; case T_JoinInfo: _outJoinInfo(str, obj); diff --git a/src/backend/nodes/readfuncs.c b/src/backend/nodes/readfuncs.c index 5e548096ae..fc7a1b4438 100644 --- a/src/backend/nodes/readfuncs.c +++ b/src/backend/nodes/readfuncs.c @@ -7,7 +7,7 @@ * * * IDENTIFICATION - * $Header: /cvsroot/pgsql/src/backend/nodes/readfuncs.c,v 1.46 1999/02/03 21:16:18 momjian Exp $ + * $Header: /cvsroot/pgsql/src/backend/nodes/readfuncs.c,v 1.47 1999/02/04 01:46:54 momjian Exp $ * * NOTES * Most of the read functions for plan nodes are tested. (In fact, they @@ -1906,19 +1906,19 @@ _readJoinMethod() } /* ---------------- - * _readHInfo + * _readHashInfo * - * HInfo is a subclass of JoinMethod. + * HashInfo is a subclass of JoinMethod. * ---------------- */ -static HInfo * -_readHInfo() +static HashInfo * +_readHashInfo() { - HInfo *local_node; + HashInfo *local_node; char *token; int length; - local_node = makeNode(HInfo); + local_node = makeNode(HashInfo); token = lsptok(NULL, &length); /* get :hashop */ token = lsptok(NULL, &length); /* now read it */ @@ -2094,7 +2094,7 @@ parsePlanString(void) else if (!strncmp(token, "JOININFO", length)) return_value = _readJoinInfo(); else if (!strncmp(token, "HINFO", length)) - return_value = _readHInfo(); + return_value = _readHashInfo(); else if (!strncmp(token, "ITER", length)) return_value = _readIter(); else if (!strncmp(token, "QUERY", length)) diff --git a/src/backend/optimizer/README b/src/backend/optimizer/README index 0bce226a79..bf67e97e38 100644 --- a/src/backend/optimizer/README +++ b/src/backend/optimizer/README @@ -93,8 +93,9 @@ planner() Optimizer Structures -------------------- -Path - info about every way to access a relation(sequential, index) - PathOrder - info about every ordering (sort, merge of relations) -RelOptInfo - info about every relation - JoinInfo - info about join combinations -RestrictInfo - info about restrictions +RelOptInfo - info about every relation + RestrictInfo - info about restrictions + JoinInfo - info about join combinations + Path - info about every way to access a relation(sequential, index) + PathOrder - info about every ordering (sort, merge of relations) + diff --git a/src/backend/optimizer/path/hashutils.c b/src/backend/optimizer/path/hashutils.c index cf393ebb1f..4cdf4752d3 100644 --- a/src/backend/optimizer/path/hashutils.c +++ b/src/backend/optimizer/path/hashutils.c @@ -7,7 +7,7 @@ * * * IDENTIFICATION - * $Header: /cvsroot/pgsql/src/backend/optimizer/path/Attic/hashutils.c,v 1.9 1999/02/03 21:16:25 momjian Exp $ + * $Header: /cvsroot/pgsql/src/backend/optimizer/path/Attic/hashutils.c,v 1.10 1999/02/04 01:46:57 momjian Exp $ * *------------------------------------------------------------------------- */ @@ -20,7 +20,7 @@ #include "optimizer/clauses.h" -static HInfo *match_hashop_hashinfo(Oid hashop, List *hashinfo_list); +static HashInfo *match_hashop_hashinfo(Oid hashop, List *hashinfo_list); /* * group-clauses-by-hashop-- @@ -54,7 +54,7 @@ group_clauses_by_hashop(List *restrictinfo_list, */ if (hashjoinop) { - HInfo *xhashinfo = (HInfo *) NULL; + HashInfo *xhashinfo = (HashInfo *) NULL; Expr *clause = restrictinfo->clause; Var *leftop = get_leftop(clause); Var *rightop = get_rightop(clause); @@ -77,7 +77,7 @@ group_clauses_by_hashop(List *restrictinfo_list, if (xhashinfo == NULL) { - xhashinfo = makeNode(HInfo); + xhashinfo = makeNode(HashInfo); xhashinfo->hashop = hashjoinop; xhashinfo->jmethod.jmkeys = NIL; @@ -105,21 +105,21 @@ group_clauses_by_hashop(List *restrictinfo_list, * Returns the node if it exists. * */ -static HInfo * +static HashInfo * match_hashop_hashinfo(Oid hashop, List *hashinfo_list) { Oid key = 0; - HInfo *xhashinfo = (HInfo *) NULL; + HashInfo *xhashinfo = (HashInfo *) NULL; List *i = NIL; foreach(i, hashinfo_list) { - xhashinfo = (HInfo *) lfirst(i); + xhashinfo = (HashInfo *) lfirst(i); key = xhashinfo->hashop; if (hashop == key) { /* found */ return xhashinfo; /* should be a hashinfo node ! */ } } - return (HInfo *) NIL; + return (HashInfo *) NIL; } diff --git a/src/backend/optimizer/path/joinpath.c b/src/backend/optimizer/path/joinpath.c index bd3c3b2410..991290178f 100644 --- a/src/backend/optimizer/path/joinpath.c +++ b/src/backend/optimizer/path/joinpath.c @@ -7,7 +7,7 @@ * * * IDENTIFICATION - * $Header: /cvsroot/pgsql/src/backend/optimizer/path/joinpath.c,v 1.12 1999/02/03 21:16:27 momjian Exp $ + * $Header: /cvsroot/pgsql/src/backend/optimizer/path/joinpath.c,v 1.13 1999/02/04 01:46:57 momjian Exp $ * *------------------------------------------------------------------------- */ @@ -579,7 +579,7 @@ hash_inner_and_outer(RelOptInfo * joinrel, RelOptInfo * innerrel, List *hashinfo_list) { - HInfo *xhashinfo = (HInfo *) NULL; + HashInfo *xhashinfo = (HashInfo *) NULL; List *hjoin_list = NIL; HashPath *temp_node = (HashPath *) NULL; List *i = NIL; @@ -589,7 +589,7 @@ hash_inner_and_outer(RelOptInfo * joinrel, foreach(i, hashinfo_list) { - xhashinfo = (HInfo *) lfirst(i); + xhashinfo = (HashInfo *) lfirst(i); outerkeys = extract_path_keys(((JoinMethod *) xhashinfo)->jmkeys, outerrel->targetlist, OUTER); diff --git a/src/backend/optimizer/path/prune.c b/src/backend/optimizer/path/prune.c index 908687e6d8..e65c4d7d67 100644 --- a/src/backend/optimizer/path/prune.c +++ b/src/backend/optimizer/path/prune.c @@ -7,7 +7,7 @@ * * * IDENTIFICATION - * $Header: /cvsroot/pgsql/src/backend/optimizer/path/Attic/prune.c,v 1.17 1998/09/01 04:29:44 momjian Exp $ + * $Header: /cvsroot/pgsql/src/backend/optimizer/path/Attic/prune.c,v 1.18 1999/02/04 01:46:58 momjian Exp $ * *------------------------------------------------------------------------- */ @@ -29,7 +29,7 @@ static List *prune_joinrel(RelOptInfo * rel, List *other_rels); /* * prune-joinrels-- * Removes any redundant relation entries from a list of rel nodes - * 'rel-list'. Obviosly, the first relation can't be a duplicate. + * 'rel-list'. Obviously, the first relation can't be a duplicate. * * Returns the resulting list. * @@ -59,7 +59,7 @@ prune_joinrels(List *rel_list) * */ static List * -prune_joinrel(RelOptInfo * rel, List *other_rels) +prune_joinrel(RelOptInfo *rel, List *other_rels) { List *i = NIL; List *result = NIL; @@ -135,8 +135,7 @@ prune_rel_path(RelOptInfo * rel, Path *unorderedpath) Path *cheapest = set_cheapest(rel, rel->pathlist); /* don't prune if not pruneable -- JMH, 11/23/92 */ - if (unorderedpath != cheapest - && rel->pruneable) + if (unorderedpath != cheapest && rel->pruneable) { rel->unorderedpath = (Path *) NULL; diff --git a/src/backend/optimizer/util/pathnode.c b/src/backend/optimizer/util/pathnode.c index 55aa83875e..7bdec4bb4a 100644 --- a/src/backend/optimizer/util/pathnode.c +++ b/src/backend/optimizer/util/pathnode.c @@ -7,7 +7,7 @@ * * * IDENTIFICATION - * $Header: /cvsroot/pgsql/src/backend/optimizer/util/pathnode.c,v 1.16 1999/02/03 21:16:52 momjian Exp $ + * $Header: /cvsroot/pgsql/src/backend/optimizer/util/pathnode.c,v 1.17 1999/02/04 01:46:59 momjian Exp $ * *------------------------------------------------------------------------- */ @@ -62,7 +62,7 @@ path_is_cheaper(Path *path1, Path *path2) * */ Path * -set_cheapest(RelOptInfo * parent_rel, List *pathlist) +set_cheapest(RelOptInfo *parent_rel, List *pathlist) { List *p; Path *cheapest_so_far; @@ -99,7 +99,7 @@ set_cheapest(RelOptInfo * parent_rel, List *pathlist) * */ List * -add_pathlist(RelOptInfo * parent_rel, List *unique_paths, List *new_paths) +add_pathlist(RelOptInfo *parent_rel, List *unique_paths, List *new_paths) { List *x; Path *new_path; diff --git a/src/include/nodes/nodes.h b/src/include/nodes/nodes.h index 88eea4c485..e0c45ea9e1 100644 --- a/src/include/nodes/nodes.h +++ b/src/include/nodes/nodes.h @@ -6,7 +6,7 @@ * * Copyright (c) 1994, Regents of the University of California * - * $Id: nodes.h,v 1.36 1999/02/03 20:15:46 momjian Exp $ + * $Id: nodes.h,v 1.37 1999/02/04 01:47:00 momjian Exp $ * *------------------------------------------------------------------------- */ @@ -81,7 +81,7 @@ typedef enum NodeTag T_MergeOrder, T_RestrictInfo, T_JoinMethod, - T_HInfo, + T_HashInfo, T_MInfo, T_JoinInfo, T_Iter, diff --git a/src/include/nodes/relation.h b/src/include/nodes/relation.h index f1a8fdfc3e..b0feae341f 100644 --- a/src/include/nodes/relation.h +++ b/src/include/nodes/relation.h @@ -6,7 +6,7 @@ * * Copyright (c) 1994, Regents of the University of California * - * $Id: relation.h,v 1.12 1999/02/03 20:15:46 momjian Exp $ + * $Id: relation.h,v 1.13 1999/02/04 01:47:02 momjian Exp $ * *------------------------------------------------------------------------- */ @@ -87,7 +87,7 @@ typedef struct RelOptInfo /* materialization information */ List *targetlist; - List *pathlist; + List *pathlist; /* Path structures */ struct Path *unorderedpath; struct Path *cheapestpath; bool pruneable; @@ -102,11 +102,11 @@ typedef struct RelOptInfo /* used by various scans and joins: */ Oid *ordering; /* OID of operators in sort order */ - List *restrictinfo; /* restriction clauses */ - List *joininfo; /* join clauses */ + List *restrictinfo; /* RestrictInfo structures */ + List *joininfo; /* JoinInfo structures */ List *innerjoin; List *superrels; -} RelOptInfo; +} RelOptInfo; extern Var *get_expr(TargetEntry *foo); @@ -132,7 +132,7 @@ typedef struct PathOrder { Oid *sortop; MergeOrder *merge; - } ord; + } ord; } PathOrder; typedef struct Path @@ -230,11 +230,11 @@ typedef struct JoinMethod List *clauses; } JoinMethod; -typedef struct HInfo +typedef struct HashInfo { JoinMethod jmethod; Oid hashop; -} HInfo; +} HashInfo; typedef struct MInfo { @@ -250,7 +250,7 @@ typedef struct JoinInfo bool mergejoinable; bool hashjoinable; bool inactive; -} JoinInfo; +} JoinInfo; typedef struct Iter {