AlleriaWindrunner commented on pull request #4747:
URL: 
https://github.com/apache/incubator-dolphinscheduler/pull/4747#issuecomment-780289220


   ok, i have analyzed three situations about this section code, and read the 
discussion in #4137.Considering that this is an internal scheduling system, I 
think you and GitFireMan's code are both fine. But I have a few suggestions for 
other users:
   1. The default implementation is a generic version.
   2. If you consider performance you can increase the worker node without 
having to optimize the code.
   
![image](https://user-images.githubusercontent.com/42635220/108156404-92730800-711b-11eb-9602-2525f0d4a6f0.png)


----------------------------------------------------------------
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]


Reply via email to