[ 
https://issues.apache.org/jira/browse/PHOENIX-1264?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14148713#comment-14148713
 ] 

ramkrishna.s.vasudevan commented on PHOENIX-1264:
-------------------------------------------------

bq.Instead of changing the default in individual sources (as shown below)
I have not changed for all the testcases.  The default has been updated in the 
QueryServicesTestImpl.
But these test cases need the value to be lower so that I can assert what am 
collecting.  So I think that is fine? No?
bq.Minor nit: how about "_AnalyzeTable" 
Ok. 
bq.Can we cover these checks with a new ScanUtil.isAnalyzeTable(Scan scan) call 
instead?
Fine.
bq. updateCache(table.getSchemaName().getString(), 
table.getTableName().getString(), true);
Ok I will once verify this.  

> Add StatisticsCollector to existing tables on first connection to cluster
> -------------------------------------------------------------------------
>
>                 Key: PHOENIX-1264
>                 URL: https://issues.apache.org/jira/browse/PHOENIX-1264
>             Project: Phoenix
>          Issue Type: Sub-task
>            Reporter: James Taylor
>            Assignee: ramkrishna.s.vasudevan
>         Attachments: Phoenix-1264_1.patch, Phoenix-1264_2.patch
>
>
> We'll need to add the StatisticsCollector to existing tables so that stats 
> are kept for these tables. We typically do that on first connection to the 
> cluster in ensureTableCreated when our system table is created by walking 
> through the existing Phoenix tables. We only need to do it if the system 
> table already exists (otherwise, it's a new installation).



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to