ChenSammi commented on code in PR #9036:
URL: https://github.com/apache/ozone/pull/9036#discussion_r2434997528


##########
hadoop-hdds/container-service/src/main/java/org/apache/hadoop/ozone/container/common/statemachine/commandhandler/DeleteBlocksCommandHandler.java:
##########
@@ -645,12 +647,17 @@ private void updateMetaData(KeyValueContainerData 
containerData,
 
       // update pending deletion blocks count and delete transaction ID in
       // in-memory container status

Review Comment:
   @priyeshkaratha , there is one more place that pendingDeleteBytes is 
persisted without check the feature finalized or not, 
KeyValueContainerMetadataInspector#checkAndRepair.  We can skip the 
pendingDeleteBytes check and repair if feature is not finalized. 



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