Re: [GSoC2014] Patch ALTER TABLE ... SET LOGGED

From: Christoph Berg <cb(at)df7cb(dot)de>
To: Fabrízio de Royes Mello <fabriziomello(at)gmail(dot)com>
Cc: Pgsql Hackers <pgsql-hackers(at)postgresql(dot)org>, Stephen Frost <sfrost(at)snowman(dot)net>
Subject: Re: [GSoC2014] Patch ALTER TABLE ... SET LOGGED
Date: 2014-07-21 12:38:49
Message-ID: 20140721123848.GE3669@msg.df7cb.de
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Re: Fabrízio de Royes Mello 2014-07-16 <CAFcNs+r_LMDDxJrAbWNJ3rMY0Qegwa-vv6V2SmDescOfb2dj+Q(at)mail(dot)gmail(dot)com>
> Anyway I think all is ok now. Is this ok for you?

Hi Fabrízio,

it's ok for me now, though Andres' concerns seem valid.

> > > + SET TABLESPACE <replaceable
> class="PARAMETER">new_tablespace</replaceable>
> > > RESET ( <replaceable
> class="PARAMETER">storage_parameter</replaceable> [, ... ] )
> > > INHERIT <replaceable class="PARAMETER">parent_table</replaceable>
> > > NO INHERIT <replaceable
> class="PARAMETER">parent_table</replaceable>
> > > OF <replaceable class="PARAMETER">type_name</replaceable>
> > > NOT OF
> > > OWNER TO <replaceable class="PARAMETER">new_owner</replaceable>
> > > - SET TABLESPACE <replaceable
> class="PARAMETER">new_tablespace</replaceable>
> >
> > That should get a footnote in the final commit message.
> >
>
> Sorry, I didn't understand what you meant.

I meant to say that when this patch gets committed, the commit message
might want to explain that while we are at editing this doc part, we
moved SET TABLESPACE to the place where it really should be.

> > I think we are almost there :)
>
> Yeah... thanks a lot for your help.

Welcome. I'll look forward to use this in production :)

Christoph
--
cb(at)df7cb(dot)de | http://www.df7cb.de/

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Andres Freund 2014-07-21 12:51:50 Re: [GSoC2014] Patch ALTER TABLE ... SET LOGGED
Previous Message Magnus Hagander 2014-07-21 10:24:56 Re: [bug fix] pg_ctl always uses the same event source