On Tue, Nov 26, 2024 at 4:57 PM huyajun <hu_ya...@qq.com> wrote: > The get_parameterized_baserel_size function does not differentiate for > PARTITIONED_REL and always appends the rel's own restriction clauses. > However, for PARTITIONED_REL, rel->tuples is computed in set_append_rel_size > which comes from the sum of lived childrel->rows. It is important to note > that childrel->rows is the estimated number of result tuples, meaning it > already includes the effect of the rel's own restriction clauses.
Generally speaking, rel->tuples is the number of 'raw tuples' on disk, while rel->rows is the estimated number of tuples after applying restriction clauses. In this regard, it seems that get_parameterized_baserel_size() does not do anything wrong. I think it might be better to modify set_append_rel_size() to set an appendrel's tuples to the sum of the tuples from each live child, rather than to its rows. This would also help us adjust the estimate for the number of distinct values in estimate_num_groups() for appendrels using the new formula introduced in 84f9a35e3. There were discussions as well as a patch for this about one year ago. Please see [1]. [1] https://postgr.es/m/cambws4-ocromekmtvdh3rbmuajqaqdk0qi4k6zouvponmwz...@mail.gmail.com Thanks Richard
