You are viewing a plain text version of this content. The canonical link for it is here.
Posted to derby-commits@db.apache.org by ka...@apache.org on 2011/03/03 09:52:32 UTC

svn commit: r1076559 - /db/derby/code/trunk/java/engine/org/apache/derby/impl/sql/catalog/DataDictionaryImpl.java

Author: kahatlen
Date: Thu Mar  3 08:52:32 2011
New Revision: 1076559

URL: http://svn.apache.org/viewvc?rev=1076559&view=rev
Log:
DERBY-5086: Disable istat logging by default

Modified:
    db/derby/code/trunk/java/engine/org/apache/derby/impl/sql/catalog/DataDictionaryImpl.java

Modified: db/derby/code/trunk/java/engine/org/apache/derby/impl/sql/catalog/DataDictionaryImpl.java
URL: http://svn.apache.org/viewvc/db/derby/code/trunk/java/engine/org/apache/derby/impl/sql/catalog/DataDictionaryImpl.java?rev=1076559&r1=1076558&r2=1076559&view=diff
==============================================================================
--- db/derby/code/trunk/java/engine/org/apache/derby/impl/sql/catalog/DataDictionaryImpl.java (original)
+++ db/derby/code/trunk/java/engine/org/apache/derby/impl/sql/catalog/DataDictionaryImpl.java Thu Mar  3 08:52:32 2011
@@ -435,8 +435,6 @@ public final class	DataDictionaryImpl
      */
     private boolean indexStatsUpdateDisabled;
     private boolean indexStatsUpdateLogging;
-    /** TODO: Remove this when code goes into production (i.e. a release). */
-    private boolean indexStatsUpdateLoggingExplicitlySet;
     private String indexStatsUpdateTracing;
 
 	//systemSQLNameNumber is the number used as the last digit during the previous call to getSystemSQLName.
@@ -649,10 +647,6 @@ public final class	DataDictionaryImpl
         // See if we should enable logging of index stats activities.
         indexStatsUpdateLogging = PropertyUtil.getSystemBoolean(
                 Property.STORAGE_AUTO_INDEX_STATS_LOGGING);
-        // TODO: Remove this when going into production code (i.e. a release).
-        indexStatsUpdateLoggingExplicitlySet =
-                PropertyUtil.getSystemProperty(
-                    Property.STORAGE_AUTO_INDEX_STATS_LOGGING) != null;
 
         // See if we should enable tracing of index stats activities.
         indexStatsUpdateTracing = PropertyUtil.getSystemProperty(
@@ -830,7 +824,6 @@ public final class	DataDictionaryImpl
                 if (dbEnableIndexStatsLogging != null) {
                     indexStatsUpdateLogging = Boolean.valueOf(
                             dbEnableIndexStatsLogging).booleanValue();
-                    indexStatsUpdateLoggingExplicitlySet = true;
                 }
                 // TODO: This property may go away in production code.
                 String dbEnableIndexStatsTracing =
@@ -13759,13 +13752,6 @@ public final class	DataDictionaryImpl
             return;
         }
 
-        // TODO: Remove this override after initial testing.
-        //       Unless logging has been explicitly disabled, turn it on to
-        //       make sure we have some information if things go wrong.
-        if (!indexStatsUpdateLoggingExplicitlySet) {
-            indexStatsUpdateLogging = true;
-        }
-
         indexRefresher = new IndexStatisticsDaemonImpl(
                    Monitor.getStream(), indexStatsUpdateLogging,
                    indexStatsUpdateTracing, db, authorizationDatabaseOwner,