[jira] [Commented] (FLINK-4265) Add a NoOpOperator

["ASF GitHub Bot (JIRA)" <ji...@apache.org>]

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

ASF GitHub Bot commented on FLINK-4265:
---------------------------------------

Github user StephanEwen commented on the issue:

    https://github.com/apache/flink/pull/2294
  
    @greghogan I am not sure I completely understand this. If I see it correctly, the no-op operator is in the API, but gets removed during the translation?
    
    It looks like a bit of a hacky approach to me. Can you try and explain how exactly it helps?


> Add a NoOpOperator
> ------------------
>
>                 Key: FLINK-4265
>                 URL: https://issues.apache.org/jira/browse/FLINK-4265
>             Project: Flink
>          Issue Type: New Feature
>          Components: DataSet API
>    Affects Versions: 1.1.0
>            Reporter: Greg Hogan
>            Assignee: Greg Hogan
>            Priority: Minor
>             Fix For: 1.1.0
>
>
> One recent feature of Gelly is algorithms which detect duplicated or similar computation which can be shared. My initial implementation could only reuse a {{DataSet}} result. Before committing to Flink this was updated to use a javassist {{ProxyFactory}} allowing configuration to be merged and results to be replaced. There were some issues, as identified in FLINK-4257. With a {{NoOpOperator}} we can remove the use of {{ProxyFactory}} and resolve the identified issues.
> This ticket adds a {{NoOpOperator}} which is unwound in {{OperatorTranslation.translate}}. The {{NoOpOperator}} contains a {{DataSet}} which is accessed by a getter and setter.



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