You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@phoenix.apache.org by ra...@apache.org on 2014/09/28 13:16:26 UTC
git commit: Phoenix-1296 Scan entire region when tenant-specific
table is analyzed
Repository: phoenix
Updated Branches:
refs/heads/master 92e120c5b -> 3102d8edf
Phoenix-1296 Scan entire region when tenant-specific table is analyzed
Project: http://git-wip-us.apache.org/repos/asf/phoenix/repo
Commit: http://git-wip-us.apache.org/repos/asf/phoenix/commit/3102d8ed
Tree: http://git-wip-us.apache.org/repos/asf/phoenix/tree/3102d8ed
Diff: http://git-wip-us.apache.org/repos/asf/phoenix/diff/3102d8ed
Branch: refs/heads/master
Commit: 3102d8edf4c2bd5a1080e91ccbc8b2fdcb2e9a31
Parents: 92e120c
Author: Ramkrishna <ra...@intel.com>
Authored: Sun Sep 28 16:45:35 2014 +0530
Committer: Ramkrishna <ra...@intel.com>
Committed: Sun Sep 28 16:45:35 2014 +0530
----------------------------------------------------------------------
.../phoenix/coprocessor/UngroupedAggregateRegionObserver.java | 6 ++++++
1 file changed, 6 insertions(+)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/phoenix/blob/3102d8ed/phoenix-core/src/main/java/org/apache/phoenix/coprocessor/UngroupedAggregateRegionObserver.java
----------------------------------------------------------------------
diff --git a/phoenix-core/src/main/java/org/apache/phoenix/coprocessor/UngroupedAggregateRegionObserver.java b/phoenix-core/src/main/java/org/apache/phoenix/coprocessor/UngroupedAggregateRegionObserver.java
index dae7465..508ae52 100644
--- a/phoenix-core/src/main/java/org/apache/phoenix/coprocessor/UngroupedAggregateRegionObserver.java
+++ b/phoenix-core/src/main/java/org/apache/phoenix/coprocessor/UngroupedAggregateRegionObserver.java
@@ -38,6 +38,7 @@ import org.apache.commons.logging.Log;
import org.apache.commons.logging.LogFactory;
import org.apache.hadoop.hbase.Cell;
import org.apache.hadoop.hbase.CoprocessorEnvironment;
+import org.apache.hadoop.hbase.HConstants;
import org.apache.hadoop.hbase.HRegionInfo;
import org.apache.hadoop.hbase.KeyValue;
import org.apache.hadoop.hbase.TableName;
@@ -153,6 +154,11 @@ public class UngroupedAggregateRegionObserver extends BaseScannerRegionObserver{
if(scan.getAttribute(BaseScannerRegionObserver.ANALYZE_TABLE) != null && statsTable != null) {
stats = new StatisticsCollector(statsTable, c.getEnvironment().getConfiguration());
isAnalyze = true;
+ // We are setting the start row and stop row such that it covers the entire region. As part
+ // of Phonenix-1263 we are storing the guideposts against the physical table rather than
+ // individual tenant specific tables.
+ scan.setStartRow(HConstants.EMPTY_START_ROW);
+ scan.setStopRow(HConstants.EMPTY_END_ROW);
}
if (ScanUtil.isLocalIndex(scan)) {
/*