[ https://issues.apache.org/jira/browse/PHOENIX-910?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Ted Yu updated PHOENIX-910: --------------------------- Attachment: 910-v2.txt That would be simpler. Added a comment in javadoc for new subclass of BooleanExpressionFilter. > Filter should override hasFilterRow() when filterRow() is overridden > -------------------------------------------------------------------- > > Key: PHOENIX-910 > URL: https://issues.apache.org/jira/browse/PHOENIX-910 > Project: Phoenix > Issue Type: Task > Reporter: Ted Yu > Assignee: Ted Yu > Attachments: 910-v1.txt, 910-v2.txt > > > There're 3 Filters where filterRow() is overridden but hasFilterRow() is not: > {code} > public boolean filterRow() { > ./phoenix-core/src/main/java/org/apache/phoenix/filter/MultiKeyValueComparisonFilter.java > public boolean filterRow() { > ./phoenix-core/src/main/java/org/apache/phoenix/filter/RowKeyComparisonFilter.java > public boolean filterRow() { > ./phoenix-core/src/main/java/org/apache/phoenix/filter/SingleKeyValueComparisonFilter.java > {code} > hasFilterRow() should be overridden and return true. -- This message was sent by Atlassian JIRA (v6.3.4#6332)