Hi,
On 2018/10/01 15:03, Michael Paquier wrote:
> On Thu, Aug 09, 2018 at 01:05:56PM +0900, Amit Langote wrote:
>> Attached updated patch.
>
> So, except if I am missing something, what we have here is a patch which
> has been debatted quite a bit and has semantics which look nice.
Thanks.
> Any
> objections if we move forward with this patch?
I wasn't able to respond to some of issues that Jesper brought up with the
approach taken by the latest patch whereby there is no separate
pg_partition_level function. He said that such a function would be useful
to get the information about the individual leaf partitions, but I was no
longer sure of providing such a function separately.
> +-- all tables in the tree
> +select *, pg_relation_size(relid) as size from
> pg_partition_children('ptif_test');
> + relid | parentid | level | isleaf | size
> +-------------+------------+-------+--------+-------
> + ptif_test | | 0 | f | 0
> + ptif_test0 | ptif_test | 1 | f | 0
> + ptif_test1 | ptif_test | 1 | f | 0
> + ptif_test2 | ptif_test | 1 | t | 16384
> + ptif_test01 | ptif_test0 | 2 | t | 24576
>
> One thing is that this test depends on the page size. There are already
> plan modifications if running the regress tests with a size other than
> 8kB, but I don't think that we should make that worse, so I would
> suggest to replace to use "pg_relation_size(relid) > 0" instead.
Might be a good idea, will do.
> I have moved the patch to next CF for now.
Thank you, I'll submit an updated version soon.
Regards,
Amit