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

Hitesh Shah commented on TEZ-2018:
----------------------------------

Found some more issues which probably need to be fixed before this patch can be 
committed:

{code}
} catch (Exception e) {
   String msg = "Tez UI WebService failed to start.";
  LOG.error(msg, e);
}
{code}
   - why is this a non-critical error? 

{code}
    LOG.debug("Tez UI History URL is not set");
{code}
   - this should be a warning especially if the webservices are enabled

{code}
if (LOG.isInfoEnabled()) {
{code}
  -  not needed




  
  

> Job Tracking and History URL should point to the Tez UI
> -------------------------------------------------------
>
>                 Key: TEZ-2018
>                 URL: https://issues.apache.org/jira/browse/TEZ-2018
>             Project: Apache Tez
>          Issue Type: New Feature
>          Components: UI
>            Reporter: Rohini Palaniswamy
>            Assignee: Prakash Ramachandran
>            Priority: Critical
>         Attachments: TEZ-2018.1.patch, TEZ-2018.2.patch, TEZ-2018.3.patch, 
> TEZ-2018.4.patch, TEZ-2018.wip.1.patch
>
>
>     We need to have the Job Tracking URL take users to the Tez UI if 
> yarn.timeline-service.enabled=true



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

Reply via email to