[ https://issues.apache.org/jira/browse/FLINK-1974?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14532458#comment-14532458
]
ASF GitHub Bot commented on FLINK-1974:
---------------------------------------
Github user jkirsch commented on a diff in the pull request:
https://github.com/apache/flink/pull/652#discussion_r29842575
--- Diff: flink-runtime/src/main/java/org/apache/flink/runtime/client/SerializedJobExecutionResult.java
---
@@ -63,6 +64,17 @@ public long getNetRuntime() {
return 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.
+ */
--- End diff --
Fixed .. sad that checkstyle is not catching that
> 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)
|