szilard-nemeth commented on PR #5380: URL: https://github.com/apache/hadoop/pull/5380#issuecomment-1530426310
Hi @susheel-gupta , Just took a look at your changes in this PR. I compared the old version of ParentQueue vs. AbstractParentQueue and found some differences: - the private int field with name 'numApplications' was volatile, now it is changed to AtomicInteger. What is the reason of this? - In the method called 'addApplication', the writelock was locked during incrementing numApplications, now I can see it removed. Do you know why this was necessary? - Same for method called 'removeApplication' Otherwise LGTM -- 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]
