szetszwo commented on PR #10813:
URL: https://github.com/apache/ozone/pull/10813#issuecomment-5381166576

   > ...  has synchronization problems (there are many synchronized methods but 
calling them separately will not be synchronized correctly.)
   
   @sarvekshayr , synchronization can be done by using a ReentrantReadWriteLock:
   - Whenever a thread submitting/running a job, acquire the write lock and 
check Job id.
   - When other threads getting the status, acquire the read lock.
   - Since we are using ReentrantReadWriteLock, remove `synchronized`  from all 
the methods.


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

Reply via email to