pg.ddx.io pgsql-committers@postgresql.org mailing list archivehelp / color / mirror / Atom feed
pgsql: Doc: clarify n_distinct_inherited setting 7+ messages / 1 participants [nested] [flat]
* pgsql: Doc: clarify n_distinct_inherited setting @ 2025-10-13 20:25 David Rowley <drowley@postgresql.org> 0 siblings, 0 replies; 7+ messages in thread From: David Rowley @ 2025-10-13 20:25 UTC (permalink / raw) To: pgsql-committers@lists.postgresql.org Doc: clarify n_distinct_inherited setting There was some confusion around how to adjust the n_distinct estimates for partitioned tables. Here we try and clarify that n_distinct_inherited needs to be adjusted rather than n_distinct. Also fix some slightly misleading text which was talking about table size rather than table rows, fix a grammatical error, and adjust some text which indicated that ANALYZE was performing calculations based on the n_distinct settings. Really it's the query planner that does this and ANALYZE only stores the overridden n_distinct estimate value in pg_statistic. Author: David Rowley <dgrowleyml@gmail.com> Reviewed-by: David G. Johnston <david.g.johnston@gmail.com> Reviewed-by: Chao Li <li.evan.chao@gmail.com> Backpatch-through: 13 Discussion: https://postgr.es/m/CAApHDvrL7a-ZytM1SP8Uk9nEw9bR2CPzVb+uP+bcNj=_q-ZmVw@mail.gmail.com Branch ------ master Details ------- https://git.postgresql.org/pg/commitdiff/615a0fc2f1fd30df863b0dde2c045eaab8018ec6 Modified Files -------------- doc/src/sgml/ref/alter_table.sgml | 34 ++++++++++++++++------------------ 1 file changed, 16 insertions(+), 18 deletions(-) ^ permalink raw reply [nested|flat] 7+ messages in thread
* pgsql: Doc: clarify n_distinct_inherited setting @ 2025-10-13 20:25 David Rowley <drowley@postgresql.org> 0 siblings, 0 replies; 7+ messages in thread From: David Rowley @ 2025-10-13 20:25 UTC (permalink / raw) To: pgsql-committers@lists.postgresql.org Doc: clarify n_distinct_inherited setting There was some confusion around how to adjust the n_distinct estimates for partitioned tables. Here we try and clarify that n_distinct_inherited needs to be adjusted rather than n_distinct. Also fix some slightly misleading text which was talking about table size rather than table rows, fix a grammatical error, and adjust some text which indicated that ANALYZE was performing calculations based on the n_distinct settings. Really it's the query planner that does this and ANALYZE only stores the overridden n_distinct estimate value in pg_statistic. Author: David Rowley <dgrowleyml@gmail.com> Reviewed-by: David G. Johnston <david.g.johnston@gmail.com> Reviewed-by: Chao Li <li.evan.chao@gmail.com> Backpatch-through: 13 Discussion: https://postgr.es/m/CAApHDvrL7a-ZytM1SP8Uk9nEw9bR2CPzVb+uP+bcNj=_q-ZmVw@mail.gmail.com Branch ------ REL_18_STABLE Details ------- https://git.postgresql.org/pg/commitdiff/f691e72585a3811a05e37d27e898b1d816ff9ff9 Modified Files -------------- doc/src/sgml/ref/alter_table.sgml | 34 ++++++++++++++++------------------ 1 file changed, 16 insertions(+), 18 deletions(-) ^ permalink raw reply [nested|flat] 7+ messages in thread
* pgsql: Doc: clarify n_distinct_inherited setting @ 2025-10-13 20:26 David Rowley <drowley@postgresql.org> 0 siblings, 0 replies; 7+ messages in thread From: David Rowley @ 2025-10-13 20:26 UTC (permalink / raw) To: pgsql-committers@lists.postgresql.org Doc: clarify n_distinct_inherited setting There was some confusion around how to adjust the n_distinct estimates for partitioned tables. Here we try and clarify that n_distinct_inherited needs to be adjusted rather than n_distinct. Also fix some slightly misleading text which was talking about table size rather than table rows, fix a grammatical error, and adjust some text which indicated that ANALYZE was performing calculations based on the n_distinct settings. Really it's the query planner that does this and ANALYZE only stores the overridden n_distinct estimate value in pg_statistic. Author: David Rowley <dgrowleyml@gmail.com> Reviewed-by: David G. Johnston <david.g.johnston@gmail.com> Reviewed-by: Chao Li <li.evan.chao@gmail.com> Backpatch-through: 13 Discussion: https://postgr.es/m/CAApHDvrL7a-ZytM1SP8Uk9nEw9bR2CPzVb+uP+bcNj=_q-ZmVw@mail.gmail.com Branch ------ REL_17_STABLE Details ------- https://git.postgresql.org/pg/commitdiff/f428b1231ccf39eded9db86acb72c87ad8aa0a8f Modified Files -------------- doc/src/sgml/ref/alter_table.sgml | 34 ++++++++++++++++------------------ 1 file changed, 16 insertions(+), 18 deletions(-) ^ permalink raw reply [nested|flat] 7+ messages in thread
* pgsql: Doc: clarify n_distinct_inherited setting @ 2025-10-13 20:26 David Rowley <drowley@postgresql.org> 0 siblings, 0 replies; 7+ messages in thread From: David Rowley @ 2025-10-13 20:26 UTC (permalink / raw) To: pgsql-committers@lists.postgresql.org Doc: clarify n_distinct_inherited setting There was some confusion around how to adjust the n_distinct estimates for partitioned tables. Here we try and clarify that n_distinct_inherited needs to be adjusted rather than n_distinct. Also fix some slightly misleading text which was talking about table size rather than table rows, fix a grammatical error, and adjust some text which indicated that ANALYZE was performing calculations based on the n_distinct settings. Really it's the query planner that does this and ANALYZE only stores the overridden n_distinct estimate value in pg_statistic. Author: David Rowley <dgrowleyml@gmail.com> Reviewed-by: David G. Johnston <david.g.johnston@gmail.com> Reviewed-by: Chao Li <li.evan.chao@gmail.com> Backpatch-through: 13 Discussion: https://postgr.es/m/CAApHDvrL7a-ZytM1SP8Uk9nEw9bR2CPzVb+uP+bcNj=_q-ZmVw@mail.gmail.com Branch ------ REL_16_STABLE Details ------- https://git.postgresql.org/pg/commitdiff/0a47f054ef8d5b9f2d9702ba1a5587777953fc7c Modified Files -------------- doc/src/sgml/ref/alter_table.sgml | 34 ++++++++++++++++------------------ 1 file changed, 16 insertions(+), 18 deletions(-) ^ permalink raw reply [nested|flat] 7+ messages in thread
* pgsql: Doc: clarify n_distinct_inherited setting @ 2025-10-13 20:27 David Rowley <drowley@postgresql.org> 0 siblings, 0 replies; 7+ messages in thread From: David Rowley @ 2025-10-13 20:27 UTC (permalink / raw) To: pgsql-committers@lists.postgresql.org Doc: clarify n_distinct_inherited setting There was some confusion around how to adjust the n_distinct estimates for partitioned tables. Here we try and clarify that n_distinct_inherited needs to be adjusted rather than n_distinct. Also fix some slightly misleading text which was talking about table size rather than table rows, fix a grammatical error, and adjust some text which indicated that ANALYZE was performing calculations based on the n_distinct settings. Really it's the query planner that does this and ANALYZE only stores the overridden n_distinct estimate value in pg_statistic. Author: David Rowley <dgrowleyml@gmail.com> Reviewed-by: David G. Johnston <david.g.johnston@gmail.com> Reviewed-by: Chao Li <li.evan.chao@gmail.com> Backpatch-through: 13 Discussion: https://postgr.es/m/CAApHDvrL7a-ZytM1SP8Uk9nEw9bR2CPzVb+uP+bcNj=_q-ZmVw@mail.gmail.com Branch ------ REL_15_STABLE Details ------- https://git.postgresql.org/pg/commitdiff/46086fb6812ffdc15f5277574cc0b5cbc4d81b89 Modified Files -------------- doc/src/sgml/ref/alter_table.sgml | 34 ++++++++++++++++------------------ 1 file changed, 16 insertions(+), 18 deletions(-) ^ permalink raw reply [nested|flat] 7+ messages in thread
* pgsql: Doc: clarify n_distinct_inherited setting @ 2025-10-13 20:27 David Rowley <drowley@postgresql.org> 0 siblings, 0 replies; 7+ messages in thread From: David Rowley @ 2025-10-13 20:27 UTC (permalink / raw) To: pgsql-committers@lists.postgresql.org Doc: clarify n_distinct_inherited setting There was some confusion around how to adjust the n_distinct estimates for partitioned tables. Here we try and clarify that n_distinct_inherited needs to be adjusted rather than n_distinct. Also fix some slightly misleading text which was talking about table size rather than table rows, fix a grammatical error, and adjust some text which indicated that ANALYZE was performing calculations based on the n_distinct settings. Really it's the query planner that does this and ANALYZE only stores the overridden n_distinct estimate value in pg_statistic. Author: David Rowley <dgrowleyml@gmail.com> Reviewed-by: David G. Johnston <david.g.johnston@gmail.com> Reviewed-by: Chao Li <li.evan.chao@gmail.com> Backpatch-through: 13 Discussion: https://postgr.es/m/CAApHDvrL7a-ZytM1SP8Uk9nEw9bR2CPzVb+uP+bcNj=_q-ZmVw@mail.gmail.com Branch ------ REL_14_STABLE Details ------- https://git.postgresql.org/pg/commitdiff/715eb62e30ac54963be63f625a1d8d50fa7d29b7 Modified Files -------------- doc/src/sgml/ref/alter_table.sgml | 34 ++++++++++++++++------------------ 1 file changed, 16 insertions(+), 18 deletions(-) ^ permalink raw reply [nested|flat] 7+ messages in thread
* pgsql: Doc: clarify n_distinct_inherited setting @ 2025-10-13 20:27 David Rowley <drowley@postgresql.org> 0 siblings, 0 replies; 7+ messages in thread From: David Rowley @ 2025-10-13 20:27 UTC (permalink / raw) To: pgsql-committers@lists.postgresql.org Doc: clarify n_distinct_inherited setting There was some confusion around how to adjust the n_distinct estimates for partitioned tables. Here we try and clarify that n_distinct_inherited needs to be adjusted rather than n_distinct. Also fix some slightly misleading text which was talking about table size rather than table rows, fix a grammatical error, and adjust some text which indicated that ANALYZE was performing calculations based on the n_distinct settings. Really it's the query planner that does this and ANALYZE only stores the overridden n_distinct estimate value in pg_statistic. Author: David Rowley <dgrowleyml@gmail.com> Reviewed-by: David G. Johnston <david.g.johnston@gmail.com> Reviewed-by: Chao Li <li.evan.chao@gmail.com> Backpatch-through: 13 Discussion: https://postgr.es/m/CAApHDvrL7a-ZytM1SP8Uk9nEw9bR2CPzVb+uP+bcNj=_q-ZmVw@mail.gmail.com Branch ------ REL_13_STABLE Details ------- https://git.postgresql.org/pg/commitdiff/c3b9e565137b187cd9b271fff366075020df6b1d Modified Files -------------- doc/src/sgml/ref/alter_table.sgml | 34 ++++++++++++++++------------------ 1 file changed, 16 insertions(+), 18 deletions(-) ^ permalink raw reply [nested|flat] 7+ messages in thread
end of thread, other threads:[~2025-10-13 20:27 UTC | newest] Thread overview: 7+ messages (download: mbox mbox.gz follow: Atom feed) -- links below jump to the message on this page -- 2025-10-13 20:25 pgsql: Doc: clarify n_distinct_inherited setting David Rowley <drowley@postgresql.org> 2025-10-13 20:25 pgsql: Doc: clarify n_distinct_inherited setting David Rowley <drowley@postgresql.org> 2025-10-13 20:26 pgsql: Doc: clarify n_distinct_inherited setting David Rowley <drowley@postgresql.org> 2025-10-13 20:26 pgsql: Doc: clarify n_distinct_inherited setting David Rowley <drowley@postgresql.org> 2025-10-13 20:27 pgsql: Doc: clarify n_distinct_inherited setting David Rowley <drowley@postgresql.org> 2025-10-13 20:27 pgsql: Doc: clarify n_distinct_inherited setting David Rowley <drowley@postgresql.org> 2025-10-13 20:27 pgsql: Doc: clarify n_distinct_inherited setting David Rowley <drowley@postgresql.org>
This inbox is served by DDX for PostgreSQL; see mirroring instructions for how to clone and mirror all data and code used for this inbox