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

Zhijie Shen commented on TEZ-1529:
----------------------------------

The patch looks good to me overall. Two nits:

1. In getJsonRootEntity, you may need to handle UndeclaredThrowableException 
too.

2. I think we can reuse the http client. It's not necessary one client per 
request.
{code}
540           httpClient = new Client(new URLConnectionClientHandler(new 
TimelineUrlConnectionFactory()),
541               config);
{code}

> ATS and TezClient integration  in secure kerberos enabled cluster
> -----------------------------------------------------------------
>
>                 Key: TEZ-1529
>                 URL: https://issues.apache.org/jira/browse/TEZ-1529
>             Project: Apache Tez
>          Issue Type: Bug
>            Reporter: Prakash Ramachandran
>            Assignee: Prakash Ramachandran
>            Priority: Blocker
>         Attachments: TEZ-1529.1.patch
>
>
> This is a follow up for TEZ-1495 which address ATS - TezClient integration. 
> however it does not enable it  in secure kerberos enabled cluster. 



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

Reply via email to