[jira] [Commented] (TEZ-1716) Additional ATS data for UI

["Rajesh Balamohan (JIRA)" <ji...@apache.org>]

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

Rajesh Balamohan commented on TEZ-1716:
---------------------------------------

Minor comments.

In DAGImpl.java, taskStats computation can be done in separate method to avoid code duplication?
{code}
 Map<String, Integer> taskStats = new HashMap<String, Integer>();
    ProgressBuilder progressBuilder = getDAGProgress();
    taskStats.put(ATSConstants.NUM_COMPLETED_TASKS, progressBuilder.getTotalTaskCount());
...
{code}

In DAGSubmittedEvent.java, can vertexNameIDMap be removed?


> Additional ATS data for UI
> --------------------------
>
>                 Key: TEZ-1716
>                 URL: https://issues.apache.org/jira/browse/TEZ-1716
>             Project: Apache Tez
>          Issue Type: Bug
>            Reporter: Hitesh Shah
>            Assignee: Hitesh Shah
>         Attachments: TEZ-1716.1.patch, TEZ-1716.2.patch
>
>
> Add failed and killed attempt info at DAG and Vertex Level.
> Add tez-site configuration contents to Tez App Entity
> Add task's successful attempt id in task data. 



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