You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@flink.apache.org by "Stefano Baghino (JIRA)" <ji...@apache.org> on 2016/02/16 11:33:18 UTC

[jira] [Commented] (FLINK-3412) Remove implicit conversions JavaStream / ScalaStream

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

Stefano Baghino commented on FLINK-3412:
----------------------------------------

This one looks easy, I can take care of it immediately. Shall I assign it to me [~StephanEwen]?

> Remove implicit conversions JavaStream / ScalaStream
> ----------------------------------------------------
>
>                 Key: FLINK-3412
>                 URL: https://issues.apache.org/jira/browse/FLINK-3412
>             Project: Flink
>          Issue Type: Bug
>          Components: Scala API
>    Affects Versions: 0.10.2
>            Reporter: Stephan Ewen
>             Fix For: 1.0.0
>
>
> I think the implicit conversions between the Java DataStream and the Scala DataStream are dangerous.
> Because conversions exist in both directions, it is possible to write methods that look like calling functions on the JavaStream, but instead convert it to a Scala stream and call a different method.
> I just accidentally implemented an infinite recursion that way (via two hidden implicit conversions).
> Making the conversions explicit (with a {{wrap()}} function like in the batch API, we add minimally more code internally (nothing is different for users), but avoid such accidental errors.



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