Jave-Chen opened a new pull request #1649: fix "wait(...)" should be used instead of "Thread.sleep(...)" when a lock is held URL: https://github.com/apache/incubator-dolphinscheduler/pull/1649 ## What is the purpose of the pull request fix "wait(...)" should be used instead of "Thread.sleep(...)" when a lock is held issue #1648 ## Brief change log Modify Thread.sleep(...) to this.wait(...) If Thread.sleep(...) is called when the current thread holds a lock, it could lead to performance and scalability issues, or even worse to deadlocks because the execution of the thread holding the lock is frozen. It's better to call wait(...) on the monitor object to temporarily release the lock and allow other threads to run.
---------------------------------------------------------------- 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
