Re: Partitioning with temp tables is broken - Mailing list pgsql-hackers

From Michael Paquier
Subject Re: Partitioning with temp tables is broken
Date
Msg-id 20180619075445.GD31737@paquier.xyz
Whole thread Raw
In response to Re: Partitioning with temp tables is broken  (Amit Langote <Langote_Amit_f8@lab.ntt.co.jp>)
Responses Re: Partitioning with temp tables is broken
List pgsql-hackers
On Tue, Jun 19, 2018 at 04:27:08PM +0900, Amit Langote wrote:
> Looking at what changed from my patch:
>
> -    One cannot have both temporary and permanent relations in a given
> -    partition tree.  That is, if the root partitioned table is permanent,
> -    so must be its partitions at all levels and vice versa.
> +    Mixing temporary and permanent relations in the same partition tree
> +    is not allowed.  Hence, if the root partitioned table is permanent,
> +    so must be its partitions at all levels and vice versa for temporary
> +    relations.
>
> The "vice versa" usage in my patch wasn't perhaps right to begin with, but
> the way your patch extends it make it a bit more confusing.  Maybe we
> should write it as: "... and likewise if the root partitioned table is
> temporary."

I like you wording better here.

> Except the point above about documentation, I'm fine with your patch.

Thanks.  I'll look at that again hopefully tomorrow or the day after and
address things for both HEAD and REL_10_STABLE.  From what I can see I
don't expect any major issues but an extra lookup may catch something,
and I am out of fuel for the day..
--
Michael

Attachment

pgsql-hackers by date:

Previous
From: Konstantin Knizhnik
Date:
Subject: Re: libpq compression
Next
From: Amit Langote
Date:
Subject: documenting forbidden expressions in column defaults