Repository: phoenix
Updated Branches:
  refs/heads/4.x-HBase-1.1 89e633046 -> 0f1e7e7fe


PHOENIX-3901 SpillableGroupByIT.testStatisticsAreNotWritten is failing


Project: http://git-wip-us.apache.org/repos/asf/phoenix/repo
Commit: http://git-wip-us.apache.org/repos/asf/phoenix/commit/0f1e7e7f
Tree: http://git-wip-us.apache.org/repos/asf/phoenix/tree/0f1e7e7f
Diff: http://git-wip-us.apache.org/repos/asf/phoenix/diff/0f1e7e7f

Branch: refs/heads/4.x-HBase-1.1
Commit: 0f1e7e7fe2c51da1a675de7a9a32e17df38e4722
Parents: 89e6330
Author: Samarth Jain <sama...@apache.org>
Authored: Wed May 31 13:17:52 2017 -0700
Committer: Samarth Jain <sama...@apache.org>
Committed: Wed May 31 13:17:52 2017 -0700

----------------------------------------------------------------------
 .../apache/phoenix/schema/stats/StatisticsCollectorFactory.java  | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/phoenix/blob/0f1e7e7f/phoenix-core/src/main/java/org/apache/phoenix/schema/stats/StatisticsCollectorFactory.java
----------------------------------------------------------------------
diff --git 
a/phoenix-core/src/main/java/org/apache/phoenix/schema/stats/StatisticsCollectorFactory.java
 
b/phoenix-core/src/main/java/org/apache/phoenix/schema/stats/StatisticsCollectorFactory.java
index 1fe329f..4e37e5c 100644
--- 
a/phoenix-core/src/main/java/org/apache/phoenix/schema/stats/StatisticsCollectorFactory.java
+++ 
b/phoenix-core/src/main/java/org/apache/phoenix/schema/stats/StatisticsCollectorFactory.java
@@ -24,8 +24,6 @@ import static 
org.apache.phoenix.query.QueryServicesOptions.DEFAULT_STATS_COLLEC
 import java.io.IOException;
 
 import org.apache.hadoop.hbase.coprocessor.RegionCoprocessorEnvironment;
-import org.apache.phoenix.query.QueryServices;
-import org.apache.phoenix.query.QueryServicesOptions;
 
 /**
  * Provides new {@link StatisticsCollector} instances based on configuration 
settings for a
@@ -67,7 +65,7 @@ public class StatisticsCollectorFactory {
         return (env.getConfiguration().getBoolean(STATS_COLLECTION_ENABLED,
             DEFAULT_STATS_COLLECTION_ENABLED)
             // old config left here for backward compatibility. TODO: remove 
in the next major release
-            || env.getConfiguration().getBoolean(STATS_ENABLED_ATTRIB, true)
+            && env.getConfiguration().getBoolean(STATS_ENABLED_ATTRIB, true)
             )
             && StatisticsUtil.isStatsEnabled(env.getRegionInfo().getTable());
     }

Reply via email to