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

Jason Lowe commented on YARN-7192:
----------------------------------

+1 lgtm.  I'd rather see the multi-listener support added up front.  It's such 
a standard form of listener APIs that it sticks out like a sore thumb when 
there's only one listener supported.  Adding it later has the potential of 
churning the public API adding deprecations, new configs, etc.  I'd rather just 
avoid that if it's easy to fix now.  However I also understand the desire to 
start simple with the known use-case and the YAGNI principle.  I guess we'll 
see.  ;-)


> Add a pluggable StateMachine Listener that is notified of NM Container State 
> changes
> ------------------------------------------------------------------------------------
>
>                 Key: YARN-7192
>                 URL: https://issues.apache.org/jira/browse/YARN-7192
>             Project: Hadoop YARN
>          Issue Type: Bug
>            Reporter: Arun Suresh
>            Assignee: Arun Suresh
>         Attachments: YARN-7192.001.patch, YARN-7192.002.patch, 
> YARN-7192.003.patch, YARN-7192.004.patch
>
>
> This JIRA is to add support for a plugggable class in the NodeManager that is 
> notified of changes to the Container StateMachine state and the events that 
> caused the change.
> The proposal is to modify the basic StateMachine class add support for a hook 
> that is called before and after a transition.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

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

Reply via email to