(2010/01/04 13:18), Tom Lane wrote:
Robert Haas<robertmh...@gmail.com> writes:
2010/1/3 KaiGai Kohei<kai...@ak.jp.nec.com>:
�if (number_of_attribute_origin(myrelid, oldattname)> 1)
� � �ereport(ERROR, ...);
Am I missing something?
That sounds about right to me,
It looks remarkably inefficient to me. Do you propose to search the
entire database's inheritance tree to derive that number? And do it
over again at each child table?
Yes,
The method I suggested would allow the
necessary information to be extracted during the initial search for
child tables, which we have to do anyway.
find_all_inheritors() returns a clean list which does not contain
duplicated OID of the inherited relation, so it seems to me we need
to change the function prototype but it affects other parts, or to add
a new function which also returns number of duplications, not only OIDs.
Or, we can call find_inheritance_children() in renameatt() as if
find_all_inheritors() doing except for list_concat_unique_oid().
What is the most preferable?
I don't have any preference in the way to fix the problem.
Thanks,
--
OSS Platform Development Division, NEC
KaiGai Kohei <kai...@ak.jp.nec.com>
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers
