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

Stamatis Zampetakis updated CALCITE-4932:
-----------------------------------------
    Summary: Deprecate JdbcCalc and remove JdbcCalcRule  (was: Deprecate 
JdbcCalc and JdbcCalcRule)

> Deprecate JdbcCalc and remove JdbcCalcRule
> ------------------------------------------
>
>                 Key: CALCITE-4932
>                 URL: https://issues.apache.org/jira/browse/CALCITE-4932
>             Project: Calcite
>          Issue Type: Task
>          Components: jdbc-adapter
>    Affects Versions: 1.28.0
>            Reporter: Francesco Gini
>            Assignee: Francesco Gini
>            Priority: Major
>              Labels: pull-request-available
>             Fix For: 1.29.0
>
>          Time Spent: 10m
>  Remaining Estimate: 0h
>
> The jdbc adapter includes a node and rule for {{Calc}} ( {{{}JdbcCal{}}}c a 
> {{JdbcCalcRule}} ). {{Calc}} is an operation that is introduced only later in 
> the planner to optimise certain code generation and it is not needed for the 
> jdbc adapter.
> As it stands the rule {{JdbcCalcRule}} is never triggered because the 
> {{Jdbc}} convention rules are applied before the {{Calc}} node is introduced 
> in the plan. However, if a user try to use a custom rule set they might end 
> up with a {{JdbcCalc}} which introduces problems like the one described in 
> https://issues.apache.org/jira/browse/CALCITE-4905 .
> Because {{JdbcCalc}} is not an optimisation needed for jdbc anyway we should 
> deprecate {{JdbcCalc}} and its respective rule {{{}JdbcCalcRule{}}}. The rule 
> should also be removed from {{JdbcRules.rules}}
> See ticket https://issues.apache.org/jira/browse/CALCITE-4905  for a 
> discussion around deprecating {{JdbcCalc}}



--
This message was sent by Atlassian Jira
(v8.20.1#820001)

Reply via email to