[
https://issues.apache.org/jira/browse/IGNITE-22755?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17871872#comment-17871872
]
Evgeny Stanilovsky edited comment on IGNITE-22755 at 8/8/24 8:27 AM:
---------------------------------------------------------------------
But i see that mentioned calcite issue is still in progress ?
also we need to return check into
{noformat}
public void testSubstring()
...
assertQuery(String.format("SELECT SUBSTRING('1234567', %d, %d)",
Integer.MIN_VALUE, 10L + Integer.MAX_VALUE))
.returns("1234567").check();
{noformat}
note: this can be moved into logic tests after IGNITE-22755
cause it was fixed in CALCITE-6433
was (Author: zstan):
But i see that mentioned calcite issue is still in progress ?
also we need to return check into
{noformat}
public void testSubstring()
...
assertQuery(String.format("SELECT SUBSTRING('1234567', %d, %d)",
Integer.MIN_VALUE, 10L + Integer.MAX_VALUE))
.returns("1234567").check();
{noformat}
cause it was fixed in CALCITE-6433
> Sql. Bump calcite version
> -------------------------
>
> Key: IGNITE-22755
> URL: https://issues.apache.org/jira/browse/IGNITE-22755
> Project: Ignite
> Issue Type: Improvement
> Components: sql
> Reporter: Konstantin Orlov
> Priority: Major
> Labels: ignite-3
>
> In IGNITE-22735, new copy-pasted place has been introduced to
> IgniteSqlToRelConvertor. After the very same changes will be done in apache
> calcite (CALCITE-6471), we need to bump the version and remove introduced
> code.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
