[
https://issues.apache.org/jira/browse/IGNITE-22701?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Maksim Zhuravkov updated IGNITE-22701:
--------------------------------------
Description:
ExpressionFactory evaluates values of RexLiterals manually instead of using the
code generated by RexToLitTranslator. This creates two distinct code paths to
do the same thing. Unfortunately, code generated by RexToLitTranslater can not
be reused to evaluate literals as is, because it produces the code that uses
concrete values, making it impossible (at least in its current form) to write a
method that
- that takes a RexLiteral, and returns its values
- the code is used for both RexToLitTranslator and ExpressionFactory.
We should investigate whether it is possible to use the same code both
RexToLitTranslator and ExpressionFactory.
was:
ExpressionFactory evaluates values of RexLiterals manually instead of using the
code generated by RexToLitTranslator. This creates two distinct code paths to
do the same thing.
ExpressionFactory should use code generated by RexToLitTranslator for
evaluating RexLiterals.
> Sql. Use compiled code to evaluate values of literals
> -----------------------------------------------------
>
> Key: IGNITE-22701
> URL: https://issues.apache.org/jira/browse/IGNITE-22701
> Project: Ignite
> Issue Type: Improvement
> Components: sql
> Reporter: Maksim Zhuravkov
> Priority: Major
> Labels: ignite-3
>
> ExpressionFactory evaluates values of RexLiterals manually instead of using
> the code generated by RexToLitTranslator. This creates two distinct code
> paths to do the same thing. Unfortunately, code generated by
> RexToLitTranslater can not be reused to evaluate literals as is, because it
> produces the code that uses concrete values, making it impossible (at least
> in its current form) to write a method that
> - that takes a RexLiteral, and returns its values
> - the code is used for both RexToLitTranslator and ExpressionFactory.
> We should investigate whether it is possible to use the same code both
> RexToLitTranslator and ExpressionFactory.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
