mxm commented on pull request #13042:
URL: https://github.com/apache/flink/pull/13042#issuecomment-669161031


   PR updated to address the comments. 
   
   - Added more reasoning to why this feature might be used
   - Moved the initialization code
   - I've skipped `MiniCluster` because I'm not sure this feature should be 
applied in a MiniCluster context. 
   - Refactored startup for YarnTaskExecutorRunner
   - I've renamed the configuration option name (defaults to false now)
   - Added tests to check that the feature is disabled by default
   
   


----------------------------------------------------------------
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:
us...@infra.apache.org


Reply via email to