(2010/01/25 14:08), Robert Haas wrote:
> 2010/1/24 KaiGai Kohei<kaigai(at)ak(dot)jp(dot)nec(dot)com>:
>> It seems to me the result is different from Bernd's report.
> Well, you tested something different, so you got a different answer.
> Your case doesn't have any multiple inheritance.
If it tries ALTER TABLE xxx RENAME TO on any multiple inheritance column,
this patch will raise an error when it founds the first column unable to
rename. (Of course, it takes inconsistency in table definitions, so we
need to prevent it.) It does not make sense in performance comparison.
The issue is whether we need to check pg_inherits for each recursion
level in renameatt(), or not. So, I checked the case when we try to
rename the root of inheritance tree.
Or, are you saying to test diamond-inheritance cases?
OSS Platform Development Division, NEC
KaiGai Kohei <kaigai(at)ak(dot)jp(dot)nec(dot)com>
In response to
pgsql-hackers by date
|Next:||From: Scott Bailey||Date: 2010-01-25 05:29:38|
|Subject: Re: Review: listagg aggregate|
|Previous:||From: Takahiro Itagaki||Date: 2010-01-25 05:28:12|
|Subject: Re: Mammoth in Core? |