Re: [HACKERS] Runtime Partition Pruning

On 27 February 2018 at 22:39, Amit Langote
<Langote_Amit_f8(at)lab(dot)ntt(dot)co(dot)jp> wrote:
> I've incorporated portions of 0002 and 0003 into my patch on the other
> thread (v34) posted at [1]. That is, mostly the changes around handling
> OR clauses and interface changes resulting from it.

Thanks. I was just in the middle of swapping the order of the patches
so that the OR clause patch was directly based on yours.

> Attached are revised version of your patches after the aforementioned
> rearrangements. Note that after I took out the optimizer portion of the
> 0003 patch to incorporate it into my patch (OR clause processing bits),
> not much was left in it, so I squashed it into 0002. So there are only
> 0001 and 0002.

I've locally got a patch which is significantly different to the v12
patch which moves lots of code into nodePartition.c and fixes up the
missing node read/write functions too.

> As a review comment on 0002, I think trypartitionprune is better written
> as try_partition_prune.

That no longer exists in the new version... Will post soonish, just
need to base it all on your v34 [1] now! :)

[1] https://www.postgresql.org/message-id/158f04ce-9deb-0457-ddcc-78fb73db4ebc%40lab.ntt.co.jp

--
David Rowley http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services