You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@ozone.apache.org by GitBox <gi...@apache.org> on 2022/10/09 13:59:18 UTC

[GitHub] [ozone] kaijchen commented on a diff in pull request #3814: HDDS-7302. Fix inaccurate container block count caused by missing block

kaijchen commented on code in PR #3814:
URL: https://github.com/apache/ozone/pull/3814#discussion_r990792660


##########
hadoop-hdds/container-service/src/main/java/org/apache/hadoop/ozone/container/keyvalue/statemachine/background/BlockDeletingService.java:
##########
@@ -601,4 +603,32 @@ void apply(Table<?, DeletedBlocksTransaction> deleteTxnsTable,
   public BlockDeletingServiceMetrics getMetrics() {
     return metrics;
   }
+
+  /**
+   * The wrapper class of the result of deleting transactions.
+   */
+  private class DeleteTransactionStats {

Review Comment:
   This class should be static.
   
   ```suggestion
     private static class DeleteTransactionStats {
   ```



-- 
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: issues-unsubscribe@ozone.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org