swamirishi commented on code in PR #8546:
URL: https://github.com/apache/ozone/pull/8546#discussion_r2132663729
##########
hadoop-ozone/integration-test/src/test/java/org/apache/hadoop/fs/ozone/TestDirectoryDeletingServiceWithFSO.java:
##########
@@ -749,30 +748,29 @@ public void testDirDeletedTableCleanUpForSnapshot()
throws Exception {
assertSubPathsCount(dirDeletingService::getMovedFilesCount, 0);
assertSubPathsCount(dirDeletingService::getMovedDirsCount, 0);
assertSubPathsCount(dirDeletingService::getDeletedDirsCount, 0);
-
// Case-2) Delete dir
fs.delete(root, true);
// After delete. 5 sub files are still in keyTable.
// 4 dirs in dirTable.
assertTableRowCount(keyTable, 5);
- assertTableRowCount(dirTable, 4);
+ assertTableRowCount(dirTable, 0);
// KeyDeletingService and DirectoryDeletingService will not
// clean up because the paths are part of a snapshot.
// As a result on 1 deleted dir and 3 deleted files will
Review Comment:
done
##########
hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/service/DirectoryDeletingService.java:
##########
@@ -76,31 +92,33 @@ public class DirectoryDeletingService extends
AbstractKeyDeletingService {
// Using multi thread for DirDeletion. Multiple threads would read
// from parent directory info from deleted directory table concurrently
// and send deletion requests.
- private final int dirDeletingCorePoolSize;
Review Comment:
done
--
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]
