sumitagrawl commented on code in PR #9382:
URL: https://github.com/apache/ozone/pull/9382#discussion_r2567942095
##########
hadoop-hdds/common/src/main/java/org/apache/hadoop/hdds/utils/BackgroundService.java:
##########
@@ -162,7 +165,9 @@ public void run() {
if (LOG.isDebugEnabled()) {
LOG.debug("Number of background tasks to execute : {}", tasks.size());
}
- synchronized (BackgroundService.this) {
+
+ try {
+ semaphore.acquire();
Review Comment:
background task are added to async executor, and comes out of loop. it will
be fraction of second to be in this block, and task count will be max 10s in
number. So this is not a problem.
--
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]
