Mailing-List: contact issues-help@flink.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@flink.apache.org
Date: Fri, 22 May 2015 08:11:17 +0000 (UTC)
From: "Maximilian Michels (JIRA)" <jira@apache.org>
To: issues@flink.apache.org
Message-ID: <JIRA.12831712.1432209599000.8134.1432282277085@Atlassian.JIRA>
In-Reply-To: <JIRA.12831712.1432209599000@Atlassian.JIRA>
References: <JIRA.12831712.1432209599000@Atlassian.JIRA>
 <JIRA.12831712.1432209599630@arcas>
Subject: [jira] [Commented] (FLINK-2070) Confusing methods print() that
 print on client vs on TaskManager
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit


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

Maximilian Michels commented on FLINK-2070:
-------------------------------------------

+1 for {{printOnTaskManager()}}

I can see printing on the task manager {{out}} being useful for debugging purposes. I'm not sure if the method should be in {{DataSet}} but rather in some Util package with an explanation in the documentation.

> Confusing methods print() that print on client vs on TaskManager
> ----------------------------------------------------------------
>
>                 Key: FLINK-2070
>                 URL: https://issues.apache.org/jira/browse/FLINK-2070
>             Project: Flink
>          Issue Type: Bug
>          Components: Core
>    Affects Versions: 0.9
>            Reporter: Stephan Ewen
>             Fix For: 0.9
>
>
> With the {{print()}} method printing on the client, the {{print(sourceIentified)}} method becomes confusing, as it has the same name, but prints on the taskManager (into its "out" files) and executes lazily.
> We should clarify the confusion by picking a more descriptive name, like {{printOnTaskManager()}}.
> I am not sure how common the use case to print into the TaskManager {{out}} files is. We could remove that method and point to the {{PrintingOutputFormat}} for that.


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