Re: Weird behaviour with ALTER TABLE ... SET TABLESPACE ... statement

Heikki Linnakangas a écrit :
> Tom Lane wrote:
>> Heikki Linnakangas <heikki.linnakangas@enterprisedb.com> writes:
>>> Yeah, seems like we need to allocate a new relfilenode in the new
>>> tablespace.
>>
>> I looked into tablecmds.c and verified that ATExecSetTableSpace doesn't
>> worry about selecting a new relfilenode.  I'm also noticing a number of
>> permissions-type checks that seem like they'd better be done in
>> ATPrepSetTableSpace, because we don't go through ATExecSetTableSpace
>> if the table requires rewriting for other reasons.
> 
> The same tests are performed in the rewriting code path in
> ATRewriteTables() and in heap_create_with_catalog().
> 
> I fixed the relfilenode allocation in 8.1-HEAD. Doesn't seem worth
> fixing in 8.0, because GetNewRelFileNode() didn't exist before 8.1, so
> we couldn't check for collisions anyway.
> 

Thanks. It works great!


-- 
Guillaume.http://www.postgresqlfr.orghttp://dalibo.com