diff options
| author | Alvaro Herrera <alvherre@alvh.no-ip.org> | 2017-11-28 23:25:47 -0300 |
|---|---|---|
| committer | Alvaro Herrera <alvherre@alvh.no-ip.org> | 2017-11-28 23:40:11 -0300 |
| commit | 414cd434ff681e5f499803458eae9d5bb32372a9 (patch) | |
| tree | 4c076c7b68c1269e713c49c229e12402e8e39c91 /src | |
| parent | cd482f295150f8311290b5234873917f2172a34a (diff) | |
| download | postgresql-414cd434ff681e5f499803458eae9d5bb32372a9.tar.gz postgresql-414cd434ff681e5f499803458eae9d5bb32372a9.zip | |
Fix extstat collection when no stats are produced for a column
This is a mistakenly placed conditional in bf2a691e02d7.
Reported by Justin Pryzby
Discussion: https://postgr.es/m/20171117214352.GE25796@telsasoft.com
Diffstat (limited to 'src')
| -rw-r--r-- | src/backend/statistics/extended_stats.c | 17 |
1 files changed, 9 insertions, 8 deletions
diff --git a/src/backend/statistics/extended_stats.c b/src/backend/statistics/extended_stats.c index db4987bde38..eeed56ff0aa 100644 --- a/src/backend/statistics/extended_stats.c +++ b/src/backend/statistics/extended_stats.c @@ -95,15 +95,16 @@ BuildRelationExtStatistics(Relation onerel, double totalrows, */ stats = lookup_var_attr_stats(onerel, stat->columns, natts, vacattrstats); - if (!stats && !IsAutoVacuumWorkerProcess()) + if (!stats) { - ereport(WARNING, - (errcode(ERRCODE_INVALID_OBJECT_DEFINITION), - errmsg("statistics object \"%s.%s\" could not be computed for relation \"%s.%s\"", - stat->schema, stat->name, - get_namespace_name(onerel->rd_rel->relnamespace), - RelationGetRelationName(onerel)), - errtable(onerel))); + if (!IsAutoVacuumWorkerProcess()) + ereport(WARNING, + (errcode(ERRCODE_INVALID_OBJECT_DEFINITION), + errmsg("statistics object \"%s.%s\" could not be computed for relation \"%s.%s\"", + stat->schema, stat->name, + get_namespace_name(onerel->rd_rel->relnamespace), + RelationGetRelationName(onerel)), + errtable(onerel))); continue; } |