You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@calcite.apache.org by "Zoltan Haindrich (JIRA)" <ji...@apache.org> on 2018/11/22 17:38:00 UTC
[jira] [Commented] (CALCITE-2695) Simplify casts which are only
widening nullability
[ https://issues.apache.org/jira/browse/CALCITE-2695?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16696150#comment-16696150 ]
Zoltan Haindrich commented on CALCITE-2695:
-------------------------------------------
opened [pr934|https://github.com/apache/calcite/pull/934]
also enabled to run simplify below cast
> Simplify casts which are only widening nullability
> --------------------------------------------------
>
> Key: CALCITE-2695
> URL: https://issues.apache.org/jira/browse/CALCITE-2695
> Project: Calcite
> Issue Type: Improvement
> Reporter: Zoltan Haindrich
> Assignee: Zoltan Haindrich
> Priority: Major
>
> There are cases in which casts are added; but they are not neccessary.
> {code}
> @Test public void testSimplifyRedundantCast() {
> RexNode expr = cast(isTrue(vBoolNotNull()), tBoolean(true));
> assertThat(expr.getType().isNullable(), is(true));
> RexNode result = simplify.simplifyUnknownAs(expr, RexUnknownAs.UNKNOWN);
> assertThat(result.getType().isNullable(), is(false));
> }
> {code}
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)