frankvicky commented on code in PR #18592:
URL: https://github.com/apache/kafka/pull/18592#discussion_r1920069080
##########
core/src/main/scala/kafka/log/LogManager.scala:
##########
@@ -354,7 +354,7 @@ class LogManager(logDirs: Seq[File],
addStrayLog(topicPartition, log)
warn(s"Loaded stray log: $logDir")
} else if (isStray(log)) {
- // Unlike Zookeeper mode, which tracks pending topic deletions under a
ZNode, KRaft is unable to prevent a topic from being recreated before every
replica has been deleted.
+ // KRaft is unable to prevent a topic from being recreated before every
replica has been deleted.
Review Comment:
I think simply deleting the zk statement is not enough, we should do a
little rephrase for it.
How about like:
```suggestion
// In KRaft, we are unable to prevent a topic from being recreated
before every replica has been deleted.
```
The rest of statement we could leave it as what it is.
--
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
