kowshik commented on a change in pull request #10742:
URL: https://github.com/apache/kafka/pull/10742#discussion_r636623193
##########
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:
Yes, because the existing identifier in the `Log` class is suffixed with
a whitespace, see:
https://github.com/apache/kafka/blob/5d4f9f917c8dc356a2d922980ba8101e0f2e7093/core/src/main/scala/kafka/log/Log.scala#L280.
--
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
