Mailing-List: contact issues-help@flink.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@flink.apache.org
Date: Thu, 7 May 2015 08:03:59 +0000 (UTC)
From: "ASF GitHub Bot (JIRA)" <jira@apache.org>
To: issues@flink.incubator.apache.org
Message-ID: <JIRA.12827304.1430837747000.33700.1430985839799@Atlassian.JIRA>
In-Reply-To: <JIRA.12827304.1430837747000@Atlassian.JIRA>
References: <JIRA.12827304.1430837747000@Atlassian.JIRA>
 <JIRA.12827304.1430837747927@arcas>
Subject: [jira] [Commented] (FLINK-1974) JobExecutionResult NetRuntime -
 document result type
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit


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

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

Github user tillrohrmann commented on a diff in the pull request:

    https://github.com/apache/flink/pull/652#discussion_r29831647
  
    --- Diff: flink-core/src/main/java/org/apache/flink/api/common/JobExecutionResult.java ---
    @@ -47,12 +48,23 @@ public JobExecutionResult(JobID jobID, long netRuntime, Map<String, Object> accu
     	 * Gets the net execution time of the job, i.e., the execution time in the parallel system,
     	 * without the pre-flight steps like the optimizer.
     	 *
    -	 * @return The net execution time.
    +	 * @return The net execution time in milliseconds.
     	 */
     	public long getNetRuntime() {
     		return this.netRuntime;
     	}
     
    +    /**
    +     * Gets the net execution time of the job, i.e., the execution time in the parallel system,
    +     * without the pre-flight steps like the optimizer in a desired unit.
    +     *
    +     * @param desiredUnit the unit of the <tt>NetRuntime</tt>
    +     * @return The net execution time in the desired unit.
    +     */
    +	public long elapsedNetRuntime(TimeUnit desiredUnit) {
    --- End diff --
    
    None that I can remember. I agree that the higher resolution is better even though if your jobs take only some milliseconds to run, then you probably need not to use Flink ;-) So lets change it to milliseconds.


> JobExecutionResult NetRuntime - document result type
> ----------------------------------------------------
>
>                 Key: FLINK-1974
>                 URL: https://issues.apache.org/jira/browse/FLINK-1974
>             Project: Flink
>          Issue Type: Improvement
>          Components: Documentation
>    Affects Versions: 0.9
>         Environment: The JobExecutionResult stores the net execution times in milliseconds, but does not say so.
> A simple fix is to add it to the JavaDoc .. a more complete fix would be to change the method name to getNetRuntimeMs or even use an auto conversion, such as getElapsedNetRuntime(Timeunit) which would autoconvert
>            Reporter: Johannes
>            Assignee: Johannes
>            Priority: Trivial
>              Labels: easyfix
>


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