On 27 February 2018 at 22:39, Amit Langote
<Langote_Amit_f8@lab.ntt.co.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