You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@ignite.apache.org by "Maksim Zhuravkov (Jira)" <ji...@apache.org> on 2023/03/01 09:05:00 UTC
[jira] [Assigned] (IGNITE-18653) Sql. The number of dynamic parameters can be checked once.
[ https://issues.apache.org/jira/browse/IGNITE-18653?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Maksim Zhuravkov reassigned IGNITE-18653:
-----------------------------------------
Assignee: Maksim Zhuravkov
> Sql. The number of dynamic parameters can be checked once.
> ----------------------------------------------------------
>
> Key: IGNITE-18653
> URL: https://issues.apache.org/jira/browse/IGNITE-18653
> Project: Ignite
> Issue Type: Improvement
> Components: sql
> Reporter: Maksim Zhuravkov
> Assignee: Maksim Zhuravkov
> Priority: Minor
> Labels: calcite, calcite3-required, ignite-3
> Fix For: 3.0.0-beta2
>
>
> Since Apache Calcite's SqlParser maintains the number of dynamic parameters internally (as a protected field) it is possible to simplify the check that ensures that the number of dynamic parameters in a query is equal to the number of provided parameters.
> 1) See how that check is implemented for an SqlExplain in IgniteSqlValidator::validate - it simply checks the number of dynamic parameters in a query. We can implement that check in similar fashion for other queries.
> 2) Remove the redundant array of bound checks for dynamic parameters in IgniteSqlValidator since they are going to be no longer necessary after this change.
>
--
This message was sent by Atlassian Jira
(v8.20.10#820010)