hi ,all I noticed that there is a ReentrantLock to protect workerHostWeights field which is already thread-safe ConcurrentHashMap. I think this is not necessary ,right ? If not ,why? One PR created to remove the unnecessary lock , if the lock can be removed ,who can help to review and merge this PR ? https://github.com/apache/incubator-dolphinscheduler/pull/2730
