georgew5656 commented on PR #14643: URL: https://github.com/apache/druid/pull/14643#issuecomment-1648473862
@kfaraz @YongGang i think it it makes sense to revert removing the locks since this is a valid synchronization use case (the main thread inserting into the map has to happen before the worker thread reads from the map). i think we can keep the non synchronized getRunningTasks/getKnownTasks/getPendingTasks from the other PR though. -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
