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

Sreenath Somarajapuram commented on TEZ-2523:
---------------------------------------------

+1 LGTM
UI Working as expected.

Hope we can fully rely on the id format.

> Tez UI: applicationId can be derived from dagid/vertexid
> --------------------------------------------------------
>
>                 Key: TEZ-2523
>                 URL: https://issues.apache.org/jira/browse/TEZ-2523
>             Project: Apache Tez
>          Issue Type: Bug
>          Components: UI
>            Reporter: Prakash Ramachandran
>            Assignee: Prakash Ramachandran
>         Attachments: TEZ-2523.1.patch
>
>
> currently the applicationId for the models dag/vertex is picked up from 
> primary filter data for the dag/vertex json. deriving this from the 
> dagid/vertexid on the models has the below benefits.
> * ensures applicationId is not null (in some corner cases this causes 
> exception in store.find)
> * makes the ui backward compatible (0.5).
> * allows to remove the appid from primary filter (TEZ-2485)



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

Reply via email to