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

Thomas Tauber-Marshall commented on KUDU-2158:
----------------------------------------------

So I've implemented the suggestion here and it works.

The one problem that I'm having is that from what I can tell, 
KuduScanner::AddConjunctPredicate() takes a KuduPredicate, which are 
constructed using KuduTable::NewComparisonPredicate(), which takes a KuduValue, 
which are constructed with static methods in KuduValue, but there's no static 
method for constructing a KuduValue for a timestamp value.

(there also aren't static KuduValue constructors for the various int types, but 
I assume its fine to just use the int64_t one for the smaller types)

> AddUpperBound for KuduScanner
> -----------------------------
>
>                 Key: KUDU-2158
>                 URL: https://issues.apache.org/jira/browse/KUDU-2158
>             Project: Kudu
>          Issue Type: Improvement
>            Reporter: Thomas Tauber-Marshall
>            Priority: Critical
>             Fix For: 1.4.0
>
>
> There's work currently ongoing on the Impala side to support min-max runtime 
> filters. To support that, it would be help if there was a 
> KuduScanner::AddUpperBound() function.
> Currently, KuduScanTokenBuilder supports AddUpperBound() and KuduScanner 
> support AddLowerBound() and AddExclusiveUpperBound(), but there's no 
> inclusive KuduScanner::AddUpperBound()



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to