On 2018/11/04 4:58, Pavel Stehule wrote: > here is a patch
Thank you, Pavel.
Here are some comments.
I mentioned it during the last review, but maybe you missed it due to the other discussion.
+ the pattern are listed. If the form <literal>\dP+</literal> + is used, a sum of size of related partitions and a description + are also displayed.
+ the pattern are listed. If the form <literal>\dPi+</literal> + is used, a sum of size of related indexes and a description + are also displayed.
+ the pattern are listed. If the form <literal>\dPt+</literal> + is used, a sum of size of related indexes and a description + are also displayed.
I suggest:
"is used, the sum of sizes of related partitions / index partitions / table partitions and associated description are also displayed."
Note that I also fixed the typo (indexes -> tables) in the last one.
Also, I wonder if we should mention in the description of \dP+ that the displayed size considers the sizes of both the tables and indexes on the individual partitions, because in the code, I see pg_total_relation_size being used. So, the text should be something like:
"is used, the sum of size of related partitions (including the table and indexes, if any) and associated description are also displayed."
should be fixed now
Code itself looks to me to be in good shape, except you seem to have also missed Michael's comment upthread:
+ /* PostgreSQL 11 has pg_partition_tree function */
/* PostgreSQL 12 has pg_partition_tree function */