You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@flink.apache.org by "ramkrishna.s.vasudevan (JIRA)" <ji...@apache.org> on 2016/03/17 18:40:33 UTC

[jira] [Commented] (FLINK-3631) CodeGenerator does not check type compatibility for equality expressions

    [ https://issues.apache.org/jira/browse/FLINK-3631?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15200008#comment-15200008 ] 

ramkrishna.s.vasudevan commented on FLINK-3631:
-----------------------------------------------

Can I take this JIRA [~fhueske]? 

> CodeGenerator does not check type compatibility for equality expressions
> ------------------------------------------------------------------------
>
>                 Key: FLINK-3631
>                 URL: https://issues.apache.org/jira/browse/FLINK-3631
>             Project: Flink
>          Issue Type: Bug
>          Components: Table API
>    Affects Versions: 1.1.0
>            Reporter: Fabian Hueske
>
> The following Table API query does not fail but produces an empty result:
> {code}
>     val env = ExecutionEnvironment.getExecutionEnvironment
>     val ds = CollectionDataSets.get3TupleDataSet(env).as('a, 'b, 'c)
>     // must fail. Field 'a is not a string.
>     ds.filter( 'a === "nope" ).collect()
> {code}
> The generated flatMap code looks like this:
> {code}
> @Override
> public void flatMap(Object _in1, org.apache.flink.util.Collector c) throws Exception {
>   org.apache.flink.api.table.Row in1 = (org.apache.flink.api.table.Row) _in1;
>   
>   java.lang.String result$17 = (java.lang.String) in1.productElement(2);
>   int result$11 = (java.lang.Integer) in1.productElement(0);
>   long result$14 = (java.lang.Long) in1.productElement(1);
>   java.lang.String result$19 = "nope";
>   
>   boolean result$21 = result$19.equals(result$11);
>   
>   if (result$21) {
>     out.setField(0, result$11);
>   	out.setField(1, result$14);
>   	out.setField(2, result$17);
>     c.collect(out);
>   }
> }
> {code}
> I would expect the query to fail due to a Integer/String type conflict.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)