summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorBruce Momjian1999-02-06 17:29:30 +0000
committerBruce Momjian1999-02-06 17:29:30 +0000
commita55376084507e8ae8b1051d259264b4c12778764 (patch)
tree73d8c6b345e65c7505b4b70fb987a45d90508e48
parentead64f317be6eae7cdff9074659f8140aea3c4d5 (diff)
Optimizer cleanup.
-rw-r--r--src/backend/nodes/equalfuncs.c3
-rw-r--r--src/backend/optimizer/path/joinutils.c6
-rw-r--r--src/backend/optimizer/path/mergeutils.c4
-rw-r--r--src/backend/optimizer/util/ordering.c18
-rw-r--r--src/backend/optimizer/util/pathnode.c6
-rw-r--r--src/include/optimizer/ordering.h6
6 files changed, 22 insertions, 21 deletions
diff --git a/src/backend/nodes/equalfuncs.c b/src/backend/nodes/equalfuncs.c
index 116d19327d..e1fa08d913 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.23 1999/02/04 01:46:54 momjian Exp $
+ * $Header: /cvsroot/pgsql/src/backend/nodes/equalfuncs.c,v 1.24 1999/02/06 17:29:25 momjian Exp $
*
*-------------------------------------------------------------------------
*/
@@ -344,7 +344,6 @@ _equalPath(Path *a, Path *b)
if (a->p_ordering.ord.sortop == NULL ||
b->p_ordering.ord.sortop == NULL)
{
-
if (a->p_ordering.ord.sortop != b->p_ordering.ord.sortop)
return false;
}
diff --git a/src/backend/optimizer/path/joinutils.c b/src/backend/optimizer/path/joinutils.c
index 411efe94fb..899da28ff0 100644
--- a/src/backend/optimizer/path/joinutils.c
+++ b/src/backend/optimizer/path/joinutils.c
@@ -7,7 +7,7 @@
*
*
* IDENTIFICATION
- * $Header: /cvsroot/pgsql/src/backend/optimizer/path/Attic/joinutils.c,v 1.9 1999/02/03 21:16:27 momjian Exp $
+ * $Header: /cvsroot/pgsql/src/backend/optimizer/path/Attic/joinutils.c,v 1.10 1999/02/06 17:29:26 momjian Exp $
*
*-------------------------------------------------------------------------
*/
@@ -218,8 +218,8 @@ match_paths_joinkeys(List *joinkeys,
key_match = every_func(joinkeys, path->keys, which_subkey);
- if (equal_path_path_ordering(ordering,
- &path->p_ordering) &&
+ if (equal_path_ordering(ordering,
+ &path->p_ordering) &&
length(joinkeys) == length(path->keys) &&
key_match)
{
diff --git a/src/backend/optimizer/path/mergeutils.c b/src/backend/optimizer/path/mergeutils.c
index 2896c4a6d8..d06a5980f9 100644
--- a/src/backend/optimizer/path/mergeutils.c
+++ b/src/backend/optimizer/path/mergeutils.c
@@ -7,7 +7,7 @@
*
*
* IDENTIFICATION
- * $Header: /cvsroot/pgsql/src/backend/optimizer/path/Attic/mergeutils.c,v 1.12 1999/02/04 03:19:09 momjian Exp $
+ * $Header: /cvsroot/pgsql/src/backend/optimizer/path/Attic/mergeutils.c,v 1.13 1999/02/06 17:29:26 momjian Exp $
*
*-------------------------------------------------------------------------
*/
@@ -115,7 +115,7 @@ match_order_mergeinfo(PathOrder *ordering, List *mergeinfo_list)
xmergeorder = mergeinfo->m_ordering;
if ((ordering->ordtype == MERGE_ORDER &&
- equal_merge_merge_ordering(ordering->ord.merge, xmergeorder)) ||
+ equal_merge_ordering(ordering->ord.merge, xmergeorder)) ||
(ordering->ordtype == SORTOP_ORDER &&
equal_path_merge_ordering(ordering->ord.sortop, xmergeorder)))
{
diff --git a/src/backend/optimizer/util/ordering.c b/src/backend/optimizer/util/ordering.c
index 15d389e5f6..59da32deb4 100644
--- a/src/backend/optimizer/util/ordering.c
+++ b/src/backend/optimizer/util/ordering.c
@@ -7,7 +7,7 @@
*
*
* IDENTIFICATION
- * $Header: /cvsroot/pgsql/src/backend/optimizer/util/Attic/ordering.c,v 1.10 1999/02/05 19:59:30 momjian Exp $
+ * $Header: /cvsroot/pgsql/src/backend/optimizer/util/Attic/ordering.c,v 1.11 1999/02/06 17:29:27 momjian Exp $
*
*-------------------------------------------------------------------------
*/
@@ -21,13 +21,13 @@
static bool equal_sortops_order(Oid *ordering1, Oid *ordering2);
/*
- * equal-path-path-ordering--
+ * equal-path-ordering--
* Returns t iff two path orderings are equal.
*
*/
bool
-equal_path_path_ordering(PathOrder *path_ordering1,
- PathOrder *path_ordering2)
+equal_path_ordering(PathOrder *path_ordering1,
+ PathOrder *path_ordering2)
{
if (path_ordering1 == path_ordering2)
return true;
@@ -83,20 +83,22 @@ equal_path_merge_ordering(Oid *path_ordering,
}
/*
- * equal-merge-merge-ordering--
+ * equal-merge-ordering--
* Returns t iff two merge orderings are equal.
*
*/
bool
-equal_merge_merge_ordering(MergeOrder *merge_ordering1,
+equal_merge_ordering(MergeOrder *merge_ordering1,
MergeOrder *merge_ordering2)
{
return equal(merge_ordering1, merge_ordering2);
}
-/*****************************************************************************
+
+/*
+ * sortops
*
- *****************************************************************************/
+ */
/*
* equal_sort_ops_order -
diff --git a/src/backend/optimizer/util/pathnode.c b/src/backend/optimizer/util/pathnode.c
index e30421c374..2b6aff284d 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.18 1999/02/04 19:20:12 momjian Exp $
+ * $Header: /cvsroot/pgsql/src/backend/optimizer/util/pathnode.c,v 1.19 1999/02/06 17:29:27 momjian Exp $
*
*-------------------------------------------------------------------------
*/
@@ -165,8 +165,8 @@ better_path(Path *new_path, List *unique_paths, bool *noOther)
path = (Path *) lfirst(temp);
if (samekeys(path->keys, new_path->keys) &&
- equal_path_path_ordering(&path->p_ordering,
- &new_path->p_ordering))
+ equal_path_ordering(&path->p_ordering,
+ &new_path->p_ordering))
{
old_path = path;
break;
diff --git a/src/include/optimizer/ordering.h b/src/include/optimizer/ordering.h
index fb3365c1ef..7bf36d4849 100644
--- a/src/include/optimizer/ordering.h
+++ b/src/include/optimizer/ordering.h
@@ -6,7 +6,7 @@
*
* Copyright (c) 1994, Regents of the University of California
*
- * $Id: ordering.h,v 1.9 1998/09/01 04:37:12 momjian Exp $
+ * $Id: ordering.h,v 1.10 1999/02/06 17:29:30 momjian Exp $
*
*-------------------------------------------------------------------------
*/
@@ -15,11 +15,11 @@
#include <nodes/relation.h>
-extern bool equal_path_path_ordering(PathOrder *path_ordering1,
+extern bool equal_path_ordering(PathOrder *path_ordering1,
PathOrder *path_ordering2);
extern bool equal_path_merge_ordering(Oid *path_ordering,
MergeOrder *merge_ordering);
-extern bool equal_merge_merge_ordering(MergeOrder *merge_ordering1,
+extern bool equal_merge_ordering(MergeOrder *merge_ordering1,
MergeOrder *merge_ordering2);
#endif /* ORDERING_H */