mirror of https://github.com/apache/kafka.git
MINOR: Correct the wrong behavior of TestUtils#verifyTopicDeletion (#17589)
Reviewers: Chia-Ping Tsai <chia7712@gmail.com>
This commit is contained in:
parent
98b7e4deaf
commit
63cdb3602a
|
@ -84,7 +84,7 @@ import java.time.Duration
|
||||||
import java.util
|
import java.util
|
||||||
import java.util.concurrent._
|
import java.util.concurrent._
|
||||||
import java.util.concurrent.atomic.AtomicBoolean
|
import java.util.concurrent.atomic.AtomicBoolean
|
||||||
import java.util.{Arrays, Collections, Optional, Properties}
|
import java.util.{Collections, Optional, Properties}
|
||||||
import scala.annotation.nowarn
|
import scala.annotation.nowarn
|
||||||
import scala.collection.mutable.ArrayBuffer
|
import scala.collection.mutable.ArrayBuffer
|
||||||
import scala.collection.{Map, Seq, mutable}
|
import scala.collection.{Map, Seq, mutable}
|
||||||
|
@ -1294,9 +1294,11 @@ object TestUtils extends Logging {
|
||||||
waitUntilTrue(() => brokers.forall(broker =>
|
waitUntilTrue(() => brokers.forall(broker =>
|
||||||
broker.config.logDirs.forall { logDir =>
|
broker.config.logDirs.forall { logDir =>
|
||||||
topicPartitions.forall { tp =>
|
topicPartitions.forall { tp =>
|
||||||
!Arrays.asList(new File(logDir).list()).asScala.exists { partitionDirectoryName =>
|
!util.Arrays.asList(new File(logDir).list()).asScala.exists { partitionDirectoryNames =>
|
||||||
partitionDirectoryName.startsWith(tp.topic + "-" + tp.partition) &&
|
partitionDirectoryNames.exists { directoryName =>
|
||||||
partitionDirectoryName.endsWith(UnifiedLog.DeleteDirSuffix)
|
directoryName.startsWith(tp.topic + "-" + tp.partition) &&
|
||||||
|
directoryName.endsWith(UnifiedLog.DeleteDirSuffix)
|
||||||
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in New Issue