[jira] [Closed] (FLINK-573) Clean-up MapOperators in optimizer

["Fabian Hueske (JIRA)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/FLINK-573?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Fabian Hueske closed FLINK-573.
-------------------------------
       Resolution: Done
    Fix Version/s:     (was: pre-apache)
                   1.0.0

Resolved by the removal of the RecordAPI.

> Clean-up MapOperators in optimizer
> ----------------------------------
>
>                 Key: FLINK-573
>                 URL: https://issues.apache.org/jira/browse/FLINK-573
>             Project: Flink
>          Issue Type: Improvement
>            Reporter: GitHub Import
>            Priority: Minor
>              Labels: github-import
>             Fix For: 1.0.0
>
>
> This issue continues a discussion started by @aljoscha  in PR ([#544|https://github.com/stratosphere/stratosphere/issues/544] | [FLINK-544|https://issues.apache.org/jira/browse/FLINK-544]).
> MapNode is responsible for PlainMapOperatorBase and not for
> MapOperatorBase, but this is a mess right now.
> Maybe we should think about cleaning the situation a bit. CollectorMap is the same as FlatMap, only the function in the interface is called map instead of flatMap. I know that this is because of historic reasons but it is still a mess because we handle all three types through the layers. For example, some code in PactCompiler looks like this:
> ```java
> else if (c instanceof MapOperatorBase) {
> 	n = new CollectorMapNode((MapOperatorBase<?>) c);
> }
> else if (c instanceof PlainMapOperatorBase) {
> 	n = new MapNode((PlainMapOperatorBase<?>) c);
> }
> else if (c instanceof FlatMapOperatorBase) {
> 	n = new FlatMapNode((FlatMapOperatorBase<?>) c);
> }
> ```
> ---------------- Imported from GitHub ----------------
> Url: https://github.com/stratosphere/stratosphere/issues/573
> Created by: [fhueske|https://github.com/fhueske]
> Labels: enhancement, optimizer, 
> Created at: Fri Mar 14 02:11:22 CET 2014
> State: open



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