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

Hudson commented on PHOENIX-3195:
---------------------------------

FAILURE: Integrated in Jenkins build Phoenix-master #1367 (See 
[https://builds.apache.org/job/Phoenix-master/1367/])
PHOENIX-3195 Addendum. (James Taylor) (larsh: rev 
5fecde8d892fdc441aaa4e3ea8e195112da88705)
* (edit) 
phoenix-core/src/main/java/org/apache/phoenix/compile/ExpressionCompiler.java
* (edit) 
phoenix-core/src/main/java/org/apache/phoenix/compile/OrderByCompiler.java


> Slight safety improvement for using DistinctPrefixFilter
> --------------------------------------------------------
>
>                 Key: PHOENIX-3195
>                 URL: https://issues.apache.org/jira/browse/PHOENIX-3195
>             Project: Phoenix
>          Issue Type: Sub-task
>            Reporter: Lars Hofhansl
>            Assignee: Lars Hofhansl
>            Priority: Minor
>             Fix For: 4.9.0, 4.8.1
>
>         Attachments: 3195.txt, PHOENIX-3195_partial.patch
>
>
> There is a very slight improvement to when we should use the 
> DistinctPrefixFilter in its current form.
> Currently we check for the absence of WhereConditions, but what we really 
> mean to make sure that only one Cell per row is seen. Phoenix already does 
> that and used the FirstKeyOnlyFilter in that case. It's  best to use this 
> condition as well, just to be absolutely safe.



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

Reply via email to