diff options
Diffstat (limited to 'src/backend/optimizer')
-rw-r--r-- | src/backend/optimizer/path/_deadcode/predmig.c | 4 | ||||
-rw-r--r-- | src/backend/optimizer/path/_deadcode/xfunc.c | 12 | ||||
-rw-r--r-- | src/backend/optimizer/path/pathkeys.c | 16 |
3 files changed, 15 insertions, 17 deletions
diff --git a/src/backend/optimizer/path/_deadcode/predmig.c b/src/backend/optimizer/path/_deadcode/predmig.c index 1781f43db1d..462f1dc24ed 100644 --- a/src/backend/optimizer/path/_deadcode/predmig.c +++ b/src/backend/optimizer/path/_deadcode/predmig.c @@ -8,7 +8,7 @@ * * * IDENTIFICATION - * $Header: /cvsroot/pgsql/src/backend/optimizer/path/_deadcode/Attic/predmig.c,v 1.9 2001/03/22 03:59:35 momjian Exp $ + * $Header: /cvsroot/pgsql/src/backend/optimizer/path/_deadcode/Attic/predmig.c,v 1.10 2001/03/22 06:16:14 momjian Exp $ * *------------------------------------------------------------------------- */ @@ -485,7 +485,7 @@ xfunc_form_groups(Query *queryInfo, Stream root, Stream bottom) } -/* ------------------- UTILITY FUNCTIONS ------------------------- */ +/* ------------------- UTILITY FUNCTIONS ------------------------- */ /* ** xfunc_free_stream diff --git a/src/backend/optimizer/path/_deadcode/xfunc.c b/src/backend/optimizer/path/_deadcode/xfunc.c index 4f5c30d52ef..82258d7baf6 100644 --- a/src/backend/optimizer/path/_deadcode/xfunc.c +++ b/src/backend/optimizer/path/_deadcode/xfunc.c @@ -10,7 +10,7 @@ * * * IDENTIFICATION - * $Header: /cvsroot/pgsql/src/backend/optimizer/path/_deadcode/Attic/xfunc.c,v 1.15 2001/01/24 19:42:58 momjian Exp $ + * $Header: /cvsroot/pgsql/src/backend/optimizer/path/_deadcode/Attic/xfunc.c,v 1.16 2001/03/22 06:16:14 momjian Exp $ * *------------------------------------------------------------------------- */ @@ -1447,15 +1447,13 @@ xfunc_copyrel(RelOptInfo from, RelOptInfo *to) if (newnode == NULL) return false; - /* ---------------- - * copy node superclass fields - * ---------------- + /* + * copy node superclass fields */ CopyNodeFields((Node) from, (Node) newnode, alloc); - /* ---------------- - * copy remainder of node - * ---------------- + /* + * copy remainder of node */ Node_Copy(from, newnode, alloc, relids); diff --git a/src/backend/optimizer/path/pathkeys.c b/src/backend/optimizer/path/pathkeys.c index f93a027cd53..4f4b6720e1e 100644 --- a/src/backend/optimizer/path/pathkeys.c +++ b/src/backend/optimizer/path/pathkeys.c @@ -11,7 +11,7 @@ * Portions Copyright (c) 1994, Regents of the University of California * * IDENTIFICATION - * $Header: /cvsroot/pgsql/src/backend/optimizer/path/pathkeys.c,v 1.31 2001/03/22 03:59:35 momjian Exp $ + * $Header: /cvsroot/pgsql/src/backend/optimizer/path/pathkeys.c,v 1.32 2001/03/22 06:16:14 momjian Exp $ * *------------------------------------------------------------------------- */ @@ -860,13 +860,13 @@ make_pathkeys_for_mergeclauses(Query *root, /* * When we are given multiple merge clauses, it's possible that - * some clauses refer to the same vars as earlier clauses. - * There's no reason for us to specify sort keys like (A,B,A) when - * (A,B) will do --- and adding redundant sort keys makes add_path - * think that this sort order is different from ones that are - * really the same, so don't do it. Since we now have a - * canonicalized pathkey, a simple ptrMember test is sufficient to - * detect redundant keys. + * some clauses refer to the same vars as earlier clauses. There's + * no reason for us to specify sort keys like (A,B,A) when (A,B) + * will do --- and adding redundant sort keys makes add_path think + * that this sort order is different from ones that are really the + * same, so don't do it. Since we now have a canonicalized + * pathkey, a simple ptrMember test is sufficient to detect + * redundant keys. */ if (!ptrMember(pathkey, pathkeys)) pathkeys = lappend(pathkeys, pathkey); |