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

Xuan Gong commented on MAPREDUCE-6618:
--------------------------------------

[~jlowe]

bq. The close method is handed the object to cleanup, namely clientProtocol, 
but it's ignoring it even in the proposed patch. It needs to cast that to a 
YARNRunner and close that object.

This sounds like the appropriate solution for this issue.

Attached a patch to address the comments.


> YarnClientProtocolProvider leaking the YarnClient thread. 
> ----------------------------------------------------------
>
>                 Key: MAPREDUCE-6618
>                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-6618
>             Project: Hadoop Map/Reduce
>          Issue Type: Bug
>            Reporter: Xuan Gong
>            Assignee: Xuan Gong
>         Attachments: MAPREDUCE-6618.1.patch, MAPREDUCE-6618.2.patch
>
>
> YarnClientProtocolProvider creates YarnRunner which includes 
> ResourceMgrDelegate. In ResourceMgrDelegate, we would initiate and start 
> yarnclient. The yarnClient thread would be leaked due to
> {code}
>   @Override
>   public void close(ClientProtocol clientProtocol) throws IOException {
>     // nothing to do
>   }
> {code} in YarnClientProtocolProvider



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

Reply via email to