viktorsomogyi commented on code in PR #13421:
URL: https://github.com/apache/kafka/pull/13421#discussion_r1232168660


##########
core/src/main/scala/kafka/server/ReplicaAlterLogDirsThread.scala:
##########
@@ -97,6 +97,16 @@ class ReplicaAlterLogDirsThread(name: String,
     }
   }
 
+  // For ReplicaAlterLogDirsThread, if the partition is marked as failed due 
to an unknown exception and the partition fetch
+  // is suspended, the paused cleanup logic of the partition needs to be 
canceled, otherwise it will lead to serious unexpected
+  // disk usage growth.

Review Comment:
   ```suggestion
     // For ReplicaAlterLogDirsThread, if the future partition is marked as 
failed due to an unknown exception and the partition fetch
     // is suspended, the paused cleanup logic of the current partition needs 
to be resumed, otherwise it can lead to unexpected
     // disk usage growth.
   ```



-- 
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: jira-unsubscr...@kafka.apache.org

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

Reply via email to