ChenSammi commented on code in PR #5149:
URL: https://github.com/apache/ozone/pull/5149#discussion_r1319663226
##########
hadoop-hdds/container-service/src/main/java/org/apache/hadoop/ozone/container/keyvalue/statemachine/background/BlockDeletingTask.java:
##########
@@ -167,6 +192,7 @@ public ContainerBackgroundTaskResult deleteViaSchema1(
if (toDeleteBlocks.isEmpty()) {
LOG.debug("No under deletion block found in container : {}",
containerData.getContainerID());
+ blocksToDelete = 0;
Review Comment:
The reset is not necessary, can be removed.
##########
hadoop-hdds/container-service/src/main/java/org/apache/hadoop/ozone/container/keyvalue/statemachine/background/BlockDeletingTask.java:
##########
@@ -321,6 +347,7 @@ private ContainerBackgroundTaskResult
deleteViaTransactionStore(
// actually no delete transactions for the container, so reset the
// pending delete block count to the correct value of zero.
containerData.resetPendingDeleteBlockCount(meta);
+ blocksToDelete = 0;
Review Comment:
Same as above.
--
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]
