[COMMITTERS] pgsql: Standardize terminology for pg_statistic_ext entries. - Mailing list pgsql-committers

From Tom Lane
Subject [COMMITTERS] pgsql: Standardize terminology for pg_statistic_ext entries.
Date
Msg-id E1d9uvZ-0003PD-US@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Standardize terminology for pg_statistic_ext entries.

Consistently refer to such an entry as a "statistics object", not just
"statistics" or "extended statistics".  Previously we had a mismash of
terms, accompanied by utter confusion as to whether the term was
singular or plural.  That's not only grating (at least to the ear of
a native English speaker) but could be outright misleading, eg in error
messages that seemed to be referring to multiple objects where only one
could be meant.

This commit fixes the code and a lot of comments (though I may have
missed a few).  I also renamed two new SQL functions,
pg_get_statisticsextdef -> pg_get_statisticsobjdef
pg_statistic_ext_is_visible -> pg_statistics_obj_is_visible
to conform better with this terminology.

I have not touched the SGML docs other than fixing those function
names; the docs certainly need work but it seems like a separable task.

Discussion: https://postgr.es/m/22676.1494557205@sss.pgh.pa.us

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/f04c9a61468904b6815b2bc73a48878817766e0e

Modified Files
--------------
doc/src/sgml/func.sgml                       | 12 ++++----
src/backend/catalog/aclchk.c                 | 13 ++++----
src/backend/catalog/namespace.c              |  8 ++---
src/backend/catalog/objectaddress.c          | 15 +++++-----
src/backend/commands/alter.c                 |  2 +-
src/backend/commands/dropcmds.c              |  2 +-
src/backend/commands/statscmds.c             | 18 ++++++------
src/backend/optimizer/util/plancat.c         |  6 ++--
src/backend/statistics/README                |  9 +++---
src/backend/statistics/README.dependencies   |  2 +-
src/backend/statistics/dependencies.c        | 14 ++++-----
src/backend/statistics/extended_stats.c      | 44 +++++++++++++++-------------
src/backend/statistics/mvdistinct.c          |  7 +++--
src/backend/utils/adt/ruleutils.c            | 16 +++++-----
src/backend/utils/adt/selfuncs.c             | 16 +++++-----
src/backend/utils/cache/relcache.c           |  4 +--
src/bin/pg_dump/pg_dump.c                    |  2 +-
src/bin/psql/describe.c                      |  4 +--
src/bin/psql/tab-complete.c                  |  2 +-
src/include/catalog/catversion.h             |  2 +-
src/include/catalog/namespace.h              |  2 +-
src/include/catalog/pg_proc.h                |  6 ++--
src/include/catalog/pg_statistic_ext.h       | 10 ++++---
src/include/utils/acl.h                      |  2 +-
src/test/regress/expected/alter_generic.out  | 10 +++----
src/test/regress/expected/object_address.out |  4 +--
src/test/regress/expected/rules.out          |  4 +--
src/test/regress/expected/stats_ext.out      | 14 ++++-----
src/test/regress/sql/object_address.sql      |  2 +-
src/test/regress/sql/rules.sql               |  2 +-
src/test/regress/sql/stats_ext.sql           |  4 +--
31 files changed, 133 insertions(+), 125 deletions(-)


pgsql-committers by date:

Previous
From: Andrew Dunstan
Date:
Subject: [COMMITTERS] pgsql: Suppress indentation from Data::Dumper in regression tests
Next
From: Tom Lane
Date:
Subject: [COMMITTERS] pgsql: Remove no-longer-needed fields of Hash plan nodes.