Fix thinko in previous patch to always update pg_class.reltuples/relpages.

I mis-simplified the test where ANALYZE decided if it could get away
without doing anything: under the new regime, that's never allowed.  Per
bug #6068 from Jeff Janes.  Back-patch to 8.4, just like previous patch.

diff --git a/src/backend/commands/analyze.c b/src/backend/commands/analyze.c
index 26a0bc25bfb7b7f1ad3e3af3b9d1a20f9f30f401..9e5f0213803005bc378199c4c580fdfc99515323 100644 (file)
--- a/src/backend/commands/analyze.c
+++ b/src/backend/commands/analyze.c
@@ -252,7 +252,6 @@ do_analyze_rel(Relation onerel, VacuumStmt *vacstmt, bool inh)
        Relation   *Irel;
        int                     nindexes;
        bool            hasindex;
-       bool            analyzableindex;
        VacAttrStats **vacattrstats;
        AnlIndexData *indexdata;
        int                     targrows,
@@ -366,7 +365,6 @@ do_analyze_rel(Relation onerel, VacuumStmt *vacstmt, bool inh)
        }
        hasindex = (nindexes > 0);
        indexdata = NULL;
-       analyzableindex = false;
        if (hasindex)
        {
                indexdata = (AnlIndexData *) palloc0(nindexes * sizeof(AnlIndexData));
@@ -400,10 +398,7 @@ do_analyze_rel(Relation onerel, VacuumStmt *vacstmt, bool inh)
                                                thisdata->vacattrstats[tcnt] =
                                                        examine_attribute(Irel[ind], i + 1, indexkey);
                                                if (thisdata->vacattrstats[tcnt] != NULL)
-                                               {
                                                        tcnt++;
-                                                       analyzableindex = true;
-                                               }
                                        }
                                }
                                thisdata->attr_cnt = tcnt;
@@ -411,16 +406,11 @@ do_analyze_rel(Relation onerel, VacuumStmt *vacstmt, bool inh)
                }
        }
 
-       /*
-        * Quit if no analyzable columns.
-        */
-       if (attr_cnt <= 0 && !analyzableindex)
-               goto cleanup;
-
        /*
         * Determine how many rows we need to sample, using the worst case from
         * all analyzable columns.      We use a lower bound of 100 rows to avoid
-        * possible overflow in Vitter's algorithm.
+        * possible overflow in Vitter's algorithm.  (Note: that will also be
+        * the target in the corner case where there are no analyzable columns.)
         */
        targrows = 100;
        for (i = 0; i < attr_cnt; i++)
@@ -559,9 +549,6 @@ do_analyze_rel(Relation onerel, VacuumStmt *vacstmt, bool inh)
        if (!inh)
                pgstat_report_analyze(onerel, totalrows, totaldeadrows);
 
-       /* We skip to here if there were no analyzable columns */
-cleanup:
-
        /* If this isn't part of VACUUM ANALYZE, let index AMs do cleanup */
        if (!(vacstmt->options & VACOPT_VACUUM))
        {