ccding commented on a change in pull request #10742:
URL: https://github.com/apache/kafka/pull/10742#discussion_r636574778
##########
File path: core/src/main/scala/kafka/log/LogLoader.scala
##########
@@ -172,7 +173,7 @@ object LogLoader extends Logging {
private def removeTempFilesAndCollectSwapFiles(params: LoadLogParams):
Set[File] = {
def deleteIndicesIfExist(baseFile: File, suffix: String = ""): Unit = {
- info(s"${params.logIdentifier} Deleting index files with suffix $suffix
for baseFile $baseFile")
+ info(s"${params.logIdentifier}Deleting index files with suffix $suffix
for baseFile $baseFile")
Review comment:
is the change (remove blank space here) along with
https://github.com/apache/kafka/pull/10742/files#diff-54b3df71b1e0697a211d23a9018a91aef773fca0b9cbd1abafbdca6c79664930L70-R70
(add blank space there) needed? [I am not sure the best practice in Kafka
codebase]
--
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.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
