[
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)