On 30 September 2016 at 12:10, Heikki Linnakangas <hlinn...@iki.fi> wrote: > I fear that using "statistics" as the name of the new object might get a bit > awkward. "statistics" is a plural, but we use it as the name of a single > object, like "pants" or "scissors". Not sure I have any better ideas though. > "estimator"? "statistics collection"? Or perhaps it should be singular, > "statistic". I note that you actually called the system table > "pg_mv_statistic", in singular. >
I think it's OK. The functional dependency is a single statistic, but MCV lists and histograms are multiple statistics (multiple facts about the data sampled), so in general when you create one of these new objects, you are creating multiple statistics about the data. Also I find "CREATE STATISTIC" just sounds a bit clumsy compared to "CREATE STATISTICS". The convention for naming system catalogs seems to be to use the singular for tables and plural for views, so I guess we should stick with that. It doesn't seem like the end of the world that it doesn't match the user-facing syntax. A bigger concern is the use of "mv" in the name, because as has already been pointed out, this table may also in the future be used to store univariate expression and partial statistics, so I think we should drop the "mv" and go with something like pg_statistic_ext, or some other more general name. Regards, Dean -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers