Github user karanmehta93 commented on a diff in the pull request:

    https://github.com/apache/phoenix/pull/351#discussion_r218635042
  
    --- Diff: 
phoenix-core/src/it/java/org/apache/phoenix/schema/stats/StatsCollectorIT.java 
---
    @@ -38,6 +38,7 @@
     import java.util.Properties;
     import java.util.Random;
     
    +import com.google.common.collect.Lists;
    --- End diff --
    
    @twdsilva 
    I had a discussion with @BinShi-SecularBird offline. The scope of this Jira 
is limited to make sure that `UPDATE STATISTICS` runs with all versions of the 
cell (not the deleted versions, the actual number of allowed versions), since 
currently this is a feature gap (and actually a functional issue).
    We will make sure that the title of Jira is updated accordingly and a new 
Jira will be filed to discuss whether deleted rows need to handled with `UPDATE 
STATISTICS` sql or not.


---

Reply via email to