From 7431796b46e53da3d548e82928c1a18c08e936c9 Mon Sep 17 00:00:00 2001 From: Tom Lane Date: Tue, 14 Dec 1999 03:35:28 +0000 Subject: fix_parsetree_attnums was not nearly smart enough about walking parse trees. Also rewrite find_all_inheritors() in a more intelligible style. --- src/backend/commands/command.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) (limited to 'src/backend/commands/command.c') diff --git a/src/backend/commands/command.c b/src/backend/commands/command.c index c26228677e9..ddf89b8d7dd 100644 --- a/src/backend/commands/command.c +++ b/src/backend/commands/command.c @@ -7,7 +7,7 @@ * * * IDENTIFICATION - * $Header: /cvsroot/pgsql/src/backend/commands/Attic/command.c,v 1.59 1999/12/10 03:55:49 momjian Exp $ + * $Header: /cvsroot/pgsql/src/backend/commands/Attic/command.c,v 1.60 1999/12/14 03:35:20 tgl Exp $ * * NOTES * The PortalExecutorHeapMemory crap needs to be eliminated @@ -340,12 +340,11 @@ PerformAddAttribute(char *relationName, { if (inherits) { - Oid childrelid; List *child, *children; /* this routine is actually in the planner */ - children = find_all_inheritors(lconsi(myrelid, NIL), NIL); + children = find_all_inheritors(myrelid); /* * find_all_inheritors does the recursive search of the @@ -354,7 +353,8 @@ PerformAddAttribute(char *relationName, */ foreach(child, children) { - childrelid = lfirsti(child); + Oid childrelid = lfirsti(child); + if (childrelid == myrelid) continue; rel = heap_open(childrelid, AccessExclusiveLock); -- cgit v1.2.3