[ 
https://issues.apache.org/jira/browse/CALCITE-7156?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Zhen Chen resolved CALCITE-7156.
--------------------------------
    Fix Version/s: 1.41.0
         Assignee: Zhen Chen
       Resolution: Fixed

Fixed in 
https://github.com/apache/calcite/commit/c327ccba1ed13c3a03ee8e4f199040c2cdec1bd3
Thanks for the review [~mbudiu][~caican][~asolimando]

> OFFSET and FETCH in EnumerableLimit need to support BIGINT
> ----------------------------------------------------------
>
>                 Key: CALCITE-7156
>                 URL: https://issues.apache.org/jira/browse/CALCITE-7156
>             Project: Calcite
>          Issue Type: Bug
>          Components: core
>    Affects Versions: 1.40.0
>            Reporter: Zhen Chen
>            Assignee: Zhen Chen
>            Priority: Major
>              Labels: pull-request-available
>             Fix For: 1.41.0
>
>
> If OFFSET or FETCH in EnumerableLimit is BIGINT, intValue() will overflow. 
> Since CALCITE-7154 has fixed, it need to be change to longValue().
> {code:java}
>   static Expression getExpression(RexNode rexNode) {
>     if (rexNode instanceof RexDynamicParam) {
>       final RexDynamicParam param = (RexDynamicParam) rexNode;
>       return Expressions.convert_(
>           Expressions.call(DataContext.ROOT,
>               BuiltInMethod.DATA_CONTEXT_GET.method,
>               Expressions.constant("?" + param.getIndex())),
>           Integer.class);
>     } else {
>       return Expressions.constant(RexLiteral.intValue(rexNode)); <--- here
>     }
>   }
> {code}



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to