From:
Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To:
Robert Haas <robertmhaas(at)gmail(dot)com>
Cc:
KaiGai Kohei <kaigai(at)kaigai(dot)gr(dot)jp>, KaiGai Kohei <kaigai(at)ak(dot)jp(dot)nec(dot)com>,
pgsql-hackers(at)postgresql(dot)org, Thom Brown <thombrown(at)gmail(dot)com>,
Alvaro Herrera <alvherre(at)commandprompt(dot)com>
Subject:
Re: [BUG?] strange behavior in ALTER TABLE ... RENAME TO on inherited columns
Date:
2010-01-03 17:31:10
Message-ID:
5531.1262539870@sss.pgh.pa.us (view raw or flat )
Thread:
2009-11-04 06:36:15 from KaiGai Kohei <kaigai(at)ak(dot)jp(dot)nec(dot)com>
2009-11-04 13:31:36 from Alvaro Herrera <alvherre(at)commandprompt(dot)com>
2009-11-04 13:48:10 from Thom Brown <thombrown(at)gmail(dot)com>
2009-11-04 14:41:31 from Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
2009-11-05 00:57:03 from KaiGai Kohei <kaigai(at)ak(dot)jp(dot)nec(dot)com>
2009-12-17 04:30:00 from KaiGai Kohei <kaigai(at)ak(dot)jp(dot)nec(dot)com>
2009-12-30 01:38:40 from Robert Haas <robertmhaas(at)gmail(dot)com>
2010-01-02 15:16:35 from KaiGai Kohei <kaigai(at)kaigai(dot)gr(dot)jp>
2010-01-02 19:32:36 from Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
2010-01-03 14:53:41 from Robert Haas <robertmhaas(at)gmail(dot)com>
2010-01-03 17:31:10 from Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
2010-01-03 19:06:04 from Robert Haas <robertmhaas(at)gmail(dot)com>
2010-01-03 23:58:22 from KaiGai Kohei <kaigai(at)ak(dot)jp(dot)nec(dot)com>
2010-01-04 03:41:49 from Robert Haas <robertmhaas(at)gmail(dot)com>
2010-01-04 04:18:26 from Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
2010-01-04 04:52:09 from KaiGai Kohei <kaigai(at)ak(dot)jp(dot)nec(dot)com>
2010-01-04 09:55:16 from KaiGai Kohei <kaigai(at)ak(dot)jp(dot)nec(dot)com>
2010-01-14 03:43:27 from KaiGai Kohei <kaigai(at)ak(dot)jp(dot)nec(dot)com>
2010-01-14 07:04:17 from KaiGai Kohei <kaigai(at)ak(dot)jp(dot)nec(dot)com>
2010-01-23 18:45:11 from Bernd Helmle <mailings(at)oopsware(dot)de>
2010-01-24 03:29:23 from Robert Haas <robertmhaas(at)gmail(dot)com>
2010-01-24 03:48:35 from KaiGai Kohei <kaigai(at)kaigai(dot)gr(dot)jp>
2010-01-24 13:36:38 from Robert Haas <robertmhaas(at)gmail(dot)com>
2010-01-24 13:37:13 from Robert Haas <robertmhaas(at)gmail(dot)com>
2010-01-24 18:09:54 from Bernd Helmle <mailings(at)oopsware(dot)de>
2010-01-24 18:23:14 from Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
2010-01-24 18:45:33 from Bernd Helmle <mailings(at)oopsware(dot)de>
2010-01-24 19:01:10 from Bernd Helmle <mailings(at)oopsware(dot)de>
2010-01-24 19:30:30 from Robert Haas <robertmhaas(at)gmail(dot)com>
2010-01-24 23:45:38 from KaiGai Kohei <kaigai(at)ak(dot)jp(dot)nec(dot)com>
2010-01-25 02:39:21 from KaiGai Kohei <kaigai(at)ak(dot)jp(dot)nec(dot)com>
2010-01-25 05:08:35 from Robert Haas <robertmhaas(at)gmail(dot)com>
2010-01-25 05:29:12 from KaiGai Kohei <kaigai(at)ak(dot)jp(dot)nec(dot)com>
2010-01-25 15:58:14 from Robert Haas <robertmhaas(at)gmail(dot)com>
2010-01-25 16:11:22 from Bernd Helmle <mailings(at)oopsware(dot)de>
2010-01-26 01:10:58 from KaiGai Kohei <kaigai(at)ak(dot)jp(dot)nec(dot)com>
2010-01-27 08:50:19 from KaiGai Kohei <kaigai(at)ak(dot)jp(dot)nec(dot)com>
2010-01-27 14:29:06 from Robert Haas <robertmhaas(at)gmail(dot)com>
2010-01-27 15:17:45 from KaiGai Kohei <kaigai(at)kaigai(dot)gr(dot)jp>
2010-01-27 20:42:45 from Robert Haas <robertmhaas(at)gmail(dot)com>
2010-01-27 21:03:51 from Bernd Helmle <mailings(at)oopsware(dot)de>
2010-01-27 21:58:35 from Robert Haas <robertmhaas(at)gmail(dot)com>
2010-01-28 01:13:49 from KaiGai Kohei <kaigai(at)ak(dot)jp(dot)nec(dot)com>
2010-01-28 15:46:44 from Robert Haas <robertmhaas(at)gmail(dot)com>
2010-01-29 00:13:02 from KaiGai Kohei <kaigai(at)ak(dot)jp(dot)nec(dot)com>
2010-01-29 00:29:27 from Robert Haas <robertmhaas(at)gmail(dot)com>
2010-01-29 00:58:46 from KaiGai Kohei <kaigai(at)ak(dot)jp(dot)nec(dot)com>
2010-01-29 03:02:22 from KaiGai Kohei <kaigai(at)ak(dot)jp(dot)nec(dot)com>
2010-01-29 18:36:20 from Robert Haas <robertmhaas(at)gmail(dot)com>
2010-01-31 23:41:11 from KaiGai Kohei <kaigai(at)ak(dot)jp(dot)nec(dot)com>
2010-02-01 03:04:53 from KaiGai Kohei <kaigai(at)ak(dot)jp(dot)nec(dot)com>
2010-02-01 18:01:21 from Robert Haas <robertmhaas(at)gmail(dot)com>
2010-02-01 18:31:10 from Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
2010-02-01 18:40:06 from Robert Haas <robertmhaas(at)gmail(dot)com>
2010-02-01 18:55:56 from Robert Haas <robertmhaas(at)gmail(dot)com>
2010-02-01 19:03:38 from Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
2010-02-01 19:29:17 from Robert Haas <robertmhaas(at)gmail(dot)com>
2010-02-02 00:48:42 from KaiGai Kohei <kaigai(at)ak(dot)jp(dot)nec(dot)com>
2010-02-02 01:47:17 from KaiGai Kohei <kaigai(at)ak(dot)jp(dot)nec(dot)com>
2010-02-02 02:09:51 from Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
2010-02-02 02:20:26 from KaiGai Kohei <kaigai(at)ak(dot)jp(dot)nec(dot)com>
2010-02-02 02:31:01 from Robert Haas <robertmhaas(at)gmail(dot)com>
2010-02-02 02:39:07 from KaiGai Kohei <kaigai(at)ak(dot)jp(dot)nec(dot)com>
2010-02-02 02:44:20 from Robert Haas <robertmhaas(at)gmail(dot)com>
2010-02-02 02:55:17 from KaiGai Kohei <kaigai(at)ak(dot)jp(dot)nec(dot)com>
2010-02-02 14:50:53 from Robert Haas <robertmhaas(at)gmail(dot)com>
2010-02-02 23:40:45 from KaiGai Kohei <kaigai(at)ak(dot)jp(dot)nec(dot)com>
2010-02-02 02:32:01 from Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
2010-01-28 00:37:19 from KaiGai Kohei <kaigai(at)ak(dot)jp(dot)nec(dot)com>
2010-01-24 12:13:44 from Bernd Helmle <mailings(at)oopsware(dot)de>
2010-01-04 18:19:55 from Robert Haas <robertmhaas(at)gmail(dot)com>
Lists:
pgsql-hackers
Robert Haas <robertmhaas(at)gmail(dot)com> writes:
> Upthread you appeared to be endorsing what KaiGai has implemented here:
> http://archives.postgresql.org/pgsql-hackers/2009-11/msg00147.php
No, I said that forbidding conflicting renames would be a good solution.
I did not endorse any specific means of testing for such a conflict.
The one in this patch is not good enough to avoid breaking cases that
actually are perfectly OK.
It would be possible to detect the problematic cases correctly by first
descending the inheritance tree and counting the number of arrivals at
different children, and then doing it again and complaining if
attinhcount was different from the count obtained the first time.
This could probably be done by modifying find_all_inheritors to count
duplicate occurrences rather than just discarding them. Whether it's
worth it is not clear.
In practice the reasonable engineering alternatives may just be to do
what KaiGai's patch does, or to do nothing. In that case I think a good
argument can be made for the latter. Nobody has ever complained about
this from the field AFAIR; but we might get complaints if we disable
cases that used to work fine.
regards, tom lane
In response to
Responses
pgsql-hackers by date
Next :From: Andrew DunstanDate: 2010-01-03 17:44:32Subject : Re: invalid UTF-8 via pl/perlPrevious :From : Tom LaneDate : 2010-01-03 17:17:38Subject : Re: invalid UTF-8 via pl/perl
