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

Ashutosh Chauhan commented on HIVE-4888:
----------------------------------------

Patch looks good. Just for documentation purposes, we can now push:
* Filters on integral types to underlying db via directSQL for eq/lt/gt/gte/lte
* Filters on string to underlying db via directSQL for eq
* Filters on string to underlying db via DN for lt/gt/gte/lte

[~sershe] .q tests test correctness, but don't test optimization done here. 
Will writing a junit test be better here?


> listPartitionsByFilter doesn't support lt/gt/lte/gte
> ----------------------------------------------------
>
>                 Key: HIVE-4888
>                 URL: https://issues.apache.org/jira/browse/HIVE-4888
>             Project: Hive
>          Issue Type: Improvement
>            Reporter: Sergey Shelukhin
>            Assignee: Sergey Shelukhin
>         Attachments: D13101.1.patch, D13101.2.patch, D13101.3.patch, 
> D13101.4.patch, HIVE-4888.00.patch, HIVE-4888.01.patch, HIVE-4888.04.patch, 
> HIVE-4888.on-top-of-4914.patch
>
>
> Filter pushdown could be improved. Based on my experiments there's no 
> reasonable way to do it with DN 2.0, due to DN bug in substring and 
> Collection.get(int) not being implemented.
> With version as low as 2.1 we can use values.get on partition to extract 
> values to compare to. Type compatibility is an issue, but is easy for strings 
> and integral values.



--
This message was sent by Atlassian JIRA
(v6.1#6144)

Reply via email to