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

Giovanni Matteo Fumarola commented on YARN-8705:
------------------------------------------------

Few comments:

{\{threadpool}} and \{{justRecovered}} are just moved within the class itself.

I don't see any refactor for \{{HeartbeatCallBack}}. Should we place it outside?

> Refactor the UAM heartbeat thread in preparation for YARN-8696
> --------------------------------------------------------------
>
>                 Key: YARN-8705
>                 URL: https://issues.apache.org/jira/browse/YARN-8705
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>            Reporter: Botong Huang
>            Assignee: Botong Huang
>            Priority: Major
>         Attachments: YARN-8705.v1.patch, YARN-8705.v2.patch
>
>
> Refactor the UAM heartbeat thread as well as call back method in preparation 
> for YARN-8696 FederationInterceptor upgrade



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to