[jira] [Comment Edited] (TEZ-3662) Vertex Duration in 0.9 Tez UI regression from 0.7

["Sreenath Somarajapuram (JIRA)" <ji...@apache.org>]

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

Sreenath Somarajapuram edited comment on TEZ-3662 at 3/15/17 3:19 PM:
----------------------------------------------------------------------

[~jeagles] valueComputerFactory takes two paths as arguments. So I doubt if passing it as second argument to another valueComputerFactory would work. Probably you can change valueComputerFactory to accept a 3rd path. Just have to make it something like.
{code}
function valueComputerFactory(path1, path2, path3) {
  return function () {
    var value = this.get(path1);
    if(value === undefined || value === null){
      value = this.get(path2 || "");
    }
    if(value === undefined || value === null){
      value = this.get(path3 || "");
    }
    return value;
  };
}
{code}


was (Author: sreenath):
[~jeagles] valueComputerFactory takes two paths as arguments. So I doubt if passing it as second argument to another valueComputerFactory would work. Probably you can change valueComputerFactory to accept a 3rd path. Just have to make it something like.
{code}
function valueComputerFactory(path1, path2,path3) {
  return function () {
    var value = this.get(path1);
    if(value === undefined || value === null){
      value = this.get(path2 || "");
    }
    if(value === undefined || value === null){
      value = this.get(path3 || "");
    }
    return value;
  };
}
{code}

> Vertex Duration in 0.9 Tez UI regression from 0.7
> -------------------------------------------------
>
>                 Key: TEZ-3662
>                 URL: https://issues.apache.org/jira/browse/TEZ-3662
>             Project: Apache Tez
>          Issue Type: Bug
>          Components: UI
>            Reporter: Jonathan Eagles
>            Assignee: Jonathan Eagles
>         Attachments: TEZ-3662.1.patch
>
>
> In Tez UI 0.7, duration for vertex is endtime - firsttaskstarttime since vertex starttime is essentially upon vertex initialization at the beginning of the dag.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)