pg.ddx.io  pgsql-committers@postgresql.org mailing list archive  
help / 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