Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Commit2cf9cf5

Browse files
committed
Clarify use of "statistics objects" in the code
The code inconsistently used "statistic object" or "statistics" wherethe correct term, as discussed, is actually "statistics object". Thisimproves the state of the code to be more consistent.While on it, fix an incorrect error message introduced ina4d75c8. Thiserror should never happen, as the code states, but it would bemisleading.Author: Justin PryzbyReviewed-by: Álvaro Herrera, Michael PaquierDiscussion:https://postgr.es/m/20210924215827.GS831@telsasoft.comBackpatch-through: 14
1 parent2a27dba commit2cf9cf5

File tree

5 files changed

+44
-44
lines changed

5 files changed

+44
-44
lines changed

‎src/backend/commands/statscmds.c

Lines changed: 14 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -559,11 +559,11 @@ CreateStatistics(CreateStatsStmt *stmt)
559559
}
560560

561561
/*
562-
* If there are no dependencies on a column, give the statisticsan auto
563-
* dependency on the whole table. In most cases, this will be redundant,
564-
* but it might not be if the statistics expressions contain no Vars
565-
* (which might seem strange but possible). This is consistent with what
566-
* we do for indexes in index_create.
562+
* If there are no dependencies on a column, give the statisticsobject an
563+
*autodependency on the whole table. In most cases, this will be
564+
*redundant,but it might not be if the statistics expressions contain no
565+
*Vars(which might seem strange but possible). This is consistent with
566+
*whatwe do for indexes in index_create.
567567
*
568568
* XXX We intentionally don't consider the expressions before adding this
569569
* dependency, because recordDependencyOnSingleRelExpr may not create any
@@ -648,9 +648,9 @@ AlterStatistics(AlterStatsStmt *stmt)
648648
stxoid=get_statistics_object_oid(stmt->defnames,stmt->missing_ok);
649649

650650
/*
651-
* If we got here and the OID is not valid, it means the statisticsdoes
652-
* not exist, but the command specified IF EXISTS. So report this as a
653-
* simple NOTICE and we're done.
651+
* If we got here and the OID is not valid, it means the statisticsobject
652+
*doesnot exist, but the command specified IF EXISTS. So report this as
653+
*asimple NOTICE and we're done.
654654
*/
655655
if (!OidIsValid(stxoid))
656656
{
@@ -767,7 +767,7 @@ RemoveStatisticsById(Oid statsOid)
767767
}
768768

769769
/*
770-
* Select a nonconflicting name for a new statistics.
770+
* Select a nonconflicting name for a new statistics object.
771771
*
772772
* name1, name2, and label are used the same way as for makeObjectName(),
773773
* except that the label can't be NULL; digits will be appended to the label
@@ -814,9 +814,9 @@ ChooseExtendedStatisticName(const char *name1, const char *name2,
814814
}
815815

816816
/*
817-
* Generate "name2" for a new statistics given the list of column names for it
818-
* This will be passed to ChooseExtendedStatisticName along with the parent
819-
* table name and a suitable label.
817+
* Generate "name2" for a new statisticsobjectgiven the list of column
818+
*names for it.This will be passed to ChooseExtendedStatisticName along
819+
*with the parenttable name and a suitable label.
820820
*
821821
* We know that less than NAMEDATALEN characters will actually be used,
822822
* so we can truncate the result once we've generated that many.
@@ -868,8 +868,8 @@ ChooseExtendedStatisticNameAddition(List *exprs)
868868
}
869869

870870
/*
871-
* StatisticsGetRelation: given a statistics's relation OID, get the OID of
872-
* the relation it isan statistics on. Uses the system cache.
871+
* StatisticsGetRelation: given a statistics object's OID, get the OID of
872+
* the relation it isdefined on. Uses the system cache.
873873
*/
874874
Oid
875875
StatisticsGetRelation(OidstatId,boolmissing_ok)

‎src/backend/commands/tablecmds.c

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -12465,13 +12465,13 @@ RememberStatisticsForRebuilding(Oid stxoid, AlteredTableInfo *tab)
1246512465
/*
1246612466
* This de-duplication check is critical for two independent reasons: we
1246712467
* mustn't try to recreate the same statistics object twice, and if the
12468-
* statistics depends on more than one column whose type is to be altered,
12469-
* we must capture its definition string before applying any of the type
12470-
* changes. ruleutils.c will get confused if we ask again later.
12468+
* statisticsobjectdepends on more than one column whose type is to be
12469+
*altered,we must capture its definition string before applying any of
12470+
*the typechanges. ruleutils.c will get confused if we ask again later.
1247112471
*/
1247212472
if (!list_member_oid(tab->changedStatisticsOids, stxoid))
1247312473
{
12474-
/* OK, capture theindex's existing definition string */
12474+
/* OK, capture thestatistics object's existing definition string */
1247512475
char *defstring = pg_get_statisticsobjdef_string(stxoid);
1247612476

1247712477
tab->changedStatisticsOids = lappend_oid(tab->changedStatisticsOids,

‎src/backend/parser/parse_utilcmd.c

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2893,7 +2893,7 @@ transformIndexStmt(Oid relid, IndexStmt *stmt, const char *queryString)
28932893
/*
28942894
* transformStatsStmt - parse analysis for CREATE STATISTICS
28952895
*
2896-
* To avoid race conditions, it's important that this functionrely only on
2896+
* To avoid race conditions, it's important that this functionrelies only on
28972897
* the passed-in relid (and not on stmt->relation) to determine the target
28982898
* relation.
28992899
*/
@@ -2949,7 +2949,7 @@ transformStatsStmt(Oid relid, CreateStatsStmt *stmt, const char *queryString)
29492949
if (list_length(pstate->p_rtable)!=1)
29502950
ereport(ERROR,
29512951
(errcode(ERRCODE_INVALID_COLUMN_REFERENCE),
2952-
errmsg("statistics expressions can refer only to the table beingindexed")));
2952+
errmsg("statistics expressions can refer only to the table beingreferenced")));
29532953

29542954
free_parsestate(pstate);
29552955

@@ -3616,7 +3616,7 @@ transformAlterTableStmt(Oid relid, AlterTableStmt *stmt,
36163616
newcmds=lappend(newcmds,newcmd);
36173617
}
36183618

3619-
/* Append extendedstatistic objects */
3619+
/* Append extendedstatistics objects */
36203620
transformExtendedStatistics(&cxt);
36213621

36223622
/* Close rel */

‎src/backend/statistics/extended_stats.c

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -182,7 +182,7 @@ BuildRelationExtStatistics(Relation onerel, double totalrows,
182182
continue;
183183
}
184184

185-
/* compute statistics target for this statistics */
185+
/* compute statistics target for this statisticsobject*/
186186
stattarget=statext_compute_stattarget(stat->stattarget,
187187
bms_num_members(stat->columns),
188188
stats);
@@ -195,7 +195,7 @@ BuildRelationExtStatistics(Relation onerel, double totalrows,
195195
if (stattarget==0)
196196
continue;
197197

198-
/* evaluate expressions (if the statistics has any) */
198+
/* evaluate expressions (if the statisticsobjecthas any) */
199199
data=make_build_data(onerel,stat,numrows,rows,stats,stattarget);
200200

201201
/* compute statistic of each requested type */
@@ -257,9 +257,9 @@ BuildRelationExtStatistics(Relation onerel, double totalrows,
257257
* when analyzing only some of the columns, this will skip statistics objects
258258
* that would require additional columns.
259259
*
260-
* See statext_compute_stattarget for details about how we computestatistics
261-
* target for a statistics object (from the object target, attribute targets
262-
* and default statistics target).
260+
* See statext_compute_stattarget for details about how we computethe
261+
*statisticstarget for a statistics object (from the object target,
262+
*attribute targetsand default statistics target).
263263
*/
264264
int
265265
ComputeExtStatisticsRows(Relationonerel,
@@ -329,8 +329,8 @@ ComputeExtStatisticsRows(Relation onerel,
329329
*
330330
* When computing target for extended statistics objects, we consider three
331331
* places where the target may be set - the statistics object itself,
332-
* attributes the statistics is defined on, and then the default statistics
333-
* target.
332+
* attributes the statisticsobjectis defined on, and then the default
333+
*statisticstarget.
334334
*
335335
* First we look at what's set for the statistics object itself, using the
336336
* ALTER STATISTICS ... SET STATISTICS command. If we find a valid value
@@ -1789,8 +1789,8 @@ statext_mcv_clauselist_selectivity(PlannerInfo *root, List *clauses, int varReli
17891789

17901790
/*
17911791
* The clause was not estimated yet, and we've extracted either
1792-
* attnumsof expressions from it. Ignore it if it's not fully
1793-
* covered by the chosen statistics.
1792+
* attnumsor expressions from it. Ignore it if it's not fully
1793+
* covered by the chosen statistics object.
17941794
*
17951795
* We need to check both attributes and expressions, and reject if
17961796
* either is not covered.

‎src/backend/utils/adt/selfuncs.c

Lines changed: 14 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -3443,13 +3443,13 @@ estimate_num_groups(PlannerInfo *root, List *groupExprs, double input_rows,
34433443
* expression, treat it as a single variable even if it's really more
34443444
* complicated.
34453445
*
3446-
* XXX This has the consequence that if there's a statisticson the
3447-
* expression, we don't split it into individual Vars. This affects
3448-
* our selection of statistics in estimate_multivariate_ndistinct,
3449-
* because it's probably better to use more accurate estimate for each
3450-
*expression and treat them as independent, than to combine estimates
3451-
*for the extracted variables when we don't know how that relates to
3452-
* the expressions.
3446+
* XXX This has the consequence that if there's a statisticsobject on
3447+
*theexpression, we don't split it into individual Vars. This
3448+
*affectsour selection of statistics in
3449+
*estimate_multivariate_ndistinct,because it's probably better to
3450+
*use more accurate estimate for each expression and treat them as
3451+
*independent, than to combine estimates for the extracted variables
3452+
*when we don't know how that relates tothe expressions.
34533453
*/
34543454
examine_variable(root,groupexpr,0,&vardata);
34553455
if (HeapTupleIsValid(vardata.statsTuple)||vardata.isunique)
@@ -3918,7 +3918,7 @@ estimate_multivariate_ndistinct(PlannerInfo *root, RelOptInfo *rel,
39183918
if (!rel->statlist)
39193919
return false;
39203920

3921-
/* look for the ndistinct statistics matching the most vars */
3921+
/* look for the ndistinct statisticsobjectmatching the most vars */
39223922
nmatches_vars=0;/* we require at least two matches */
39233923
nmatches_exprs=0;
39243924
foreach(lc,rel->statlist)
@@ -3964,7 +3964,7 @@ estimate_multivariate_ndistinct(PlannerInfo *root, RelOptInfo *rel,
39643964
continue;
39653965
}
39663966

3967-
/* expression - see if it's in the statistics */
3967+
/* expression - see if it's in the statisticsobject*/
39683968
foreach(lc3,info->exprs)
39693969
{
39703970
Node*expr= (Node*)lfirst(lc3);
@@ -4053,7 +4053,7 @@ estimate_multivariate_ndistinct(PlannerInfo *root, RelOptInfo *rel,
40534053
if (!AttrNumberIsForUserDefinedAttr(attnum))
40544054
continue;
40554055

4056-
/* Is the variable covered by the statistics? */
4056+
/* Is the variable covered by the statistics object? */
40574057
if (!bms_is_member(attnum,matched_info->keys))
40584058
continue;
40594059

@@ -4075,7 +4075,7 @@ estimate_multivariate_ndistinct(PlannerInfo *root, RelOptInfo *rel,
40754075
if (found)
40764076
continue;
40774077

4078-
/* expression - see if it's in the statistics */
4078+
/* expression - see if it's in the statisticsobject*/
40794079
idx=0;
40804080
foreach(lc3,matched_info->exprs)
40814081
{
@@ -5252,7 +5252,7 @@ examine_variable(PlannerInfo *root, Node *node, int varRelid,
52525252
{
52535253
HeapTuplet=statext_expressions_load(info->statOid,pos);
52545254

5255-
/* Getindex's table for permission check */
5255+
/* Getstatistics object's table for permission check */
52565256
RangeTblEntry*rte;
52575257
Oiduserid;
52585258

@@ -5276,8 +5276,8 @@ examine_variable(PlannerInfo *root, Node *node, int varRelid,
52765276
/*
52775277
* For simplicity, we insist on the whole table being
52785278
* selectable, rather than trying to identify which
5279-
* column(s) the statistics depends on. Also require all
5280-
* rows to be selectable --- there must be no
5279+
* column(s) the statisticsobjectdepends on. Also
5280+
*require allrows to be selectable --- there must be no
52815281
* securityQuals from security barrier views or RLS
52825282
* policies.
52835283
*/

0 commit comments

Comments
 (0)
[8]ページ先頭
©2009-2025 Movatter.jp