You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@calcite.apache.org by "Julian Hyde (JIRA)" <ji...@apache.org> on 2016/08/31 23:51:20 UTC
[jira] [Comment Edited] (CALCITE-1220) Further extend simplify for
reducing expressions
[ https://issues.apache.org/jira/browse/CALCITE-1220?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15453765#comment-15453765 ]
Julian Hyde edited comment on CALCITE-1220 at 8/31/16 11:50 PM:
----------------------------------------------------------------
[~jcamachorodriguez], In RexUtil, why did you write
{code}
terms.remove(i);
terms.add(i, term);
{code}
instead of
{code}
terms.set(i, term);
{code}
On an ArrayList, the former are two O\(n) operations, and the latter is an O(1) operation.
was (Author: julianhyde):
[~jcamachorodriguez], In RexUtil, why did you write
{code}
terms.remove(i);
terms.add(i, term);
{code}
instead of
{code}
terms.set(i, term);
{code}
On an ArrayList, the former are two O(n) operations, and the latter is an O(1) operation.
> Further extend simplify for reducing expressions
> ------------------------------------------------
>
> Key: CALCITE-1220
> URL: https://issues.apache.org/jira/browse/CALCITE-1220
> Project: Calcite
> Issue Type: Bug
> Components: core
> Reporter: Jesus Camacho Rodriguez
> Assignee: Jesus Camacho Rodriguez
> Fix For: 1.9.0
>
>
> We would extend expression simplification, e.g.:
> - add walker to perform recursive simplification of expressions for Filter operators, and
> - extends simplification for CASE expressions to cover more cases e.g. if all return values are equal (including ELSE), CASE can be removed
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)