fix empty table now suggest 1 page instead of 0 for empty table - zero causes unwilli...

[plantuner.git] / plantuner.c

diff --git a/plantuner.c b/plantuner.c
index 050e3b7..8595463 100644 (file)
--- a/plantuner.c
+++ b/plantuner.c
@@ -32,6 +32,7 @@
 #include <fmgr.h>
 #include <miscadmin.h>
 #include <access/heapam.h>
+#include <access/xact.h>
 #include <catalog/namespace.h>
 #include <catalog/pg_class.h>
 #include <nodes/pg_list.h>
@@ -77,6 +78,22 @@ indexesAssign(const char * newval, bool doit, GucSource source, bool isDisable)
        if (!SplitIdentifierString(rawname, ',', &namelist))
                goto cleanup;
 
+       /*
+        * follow work could be done only in normal processing because of
+        * accsess to system catalog
+        */
+       if (MyBackendId == InvalidBackendId || !IsUnderPostmaster ||
+               !IsTransactionState())
+       {
+               /* reset init state */
+               if (isDisable)
+                       plantuner_disable_inited = false;
+               else
+                       plantuner_enable_inited = false;
+
+               return newval;
+       }
+
        if (doit)
        {
                nOids = list_length(namelist);
@@ -86,14 +103,6 @@ indexesAssign(const char * newval, bool doit, GucSource source, bool isDisable)
                                 (int)(sizeof(Oid) * (nOids+1)));
        }
 
-       /*
-        * follow work could be done only in normal processing because of
-        * accsess to system catalog
-        */
-       if (MyBackendId == InvalidBackendId || !IsUnderPostmaster ||
-               !IsNormalProcessingMode() || MyAuxProcType != NotAnAuxProcess)
-               return newval;
-
        if (isDisable)
                plantuner_disable_inited = true;
        else
@@ -282,7 +291,7 @@ execPlantuner(PlannerInfo *root, Oid relationObjectId, bool inhparent,
                         * estimate_rel_size() could be too pessimistic for particular
                         * workload
                         */
-                       rel->pages = 0.0;
+                       rel->pages = 1.0;
                        rel->tuples = 0.0;
                }