ZihanLi58 commented on issue #2770: [GOBBLIN-916] Make ContainerLaunchContext 
instantiation in YarnService more efficient
URL: 
https://github.com/apache/incubator-gobblin/pull/2770#issuecomment-545058459
 
 
   > Any chance we can actually verify that onStartContainerError() and 
handleContainerCompletion() are both invoked? Is it possible to subscribe to 
the eventbus defined in YarnService and handle the events emitted in each of 
these method calls?
   
   We have already emitted "HelixInstanceCompletion" event in 
handleContainerCompletion() and "ContainerStartError" event in 
onStartContainerError(). And the eventbus is used to handle container-related 
requests, which seems not related to this.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to