Skip to content

Commit

Permalink
MINOR: Remove unused compressionType parameter from TestUtils.produce…
Browse files Browse the repository at this point in the history
…Messages (apache#5569)

Reviewers: Ismael Juma <[email protected]>
  • Loading branch information
omkreddy authored and ijuma committed Aug 25, 2018
1 parent 42610e5 commit 50a145d
Showing 1 changed file with 3 additions and 7 deletions.
10 changes: 3 additions & 7 deletions core/src/test/scala/unit/kafka/utils/TestUtils.scala
Original file line number Diff line number Diff line change
Expand Up @@ -916,10 +916,7 @@ object TestUtils extends Logging {

def produceMessages(servers: Seq[KafkaServer],
records: Seq[ProducerRecord[Array[Byte], Array[Byte]]],
acks: Int = -1,
compressionType: CompressionType = CompressionType.NONE): Unit = {
val props = new Properties()
props.put(ProducerConfig.COMPRESSION_TYPE_CONFIG, compressionType.name)
acks: Int = -1): Unit = {
val producer = createProducer(TestUtils.getBrokerListStrFromServers(servers), acks = acks)
try {
val futures = records.map(producer.send)
Expand All @@ -935,11 +932,10 @@ object TestUtils extends Logging {
def generateAndProduceMessages(servers: Seq[KafkaServer],
topic: String,
numMessages: Int,
acks: Int = -1,
compressionType: CompressionType = CompressionType.NONE): Seq[String] = {
acks: Int = -1): Seq[String] = {
val values = (0 until numMessages).map(x => s"test-$x")
val records = values.map(v => new ProducerRecord[Array[Byte], Array[Byte]](topic, v.getBytes))
produceMessages(servers, records, acks, compressionType)
produceMessages(servers, records, acks)
values
}

Expand Down

0 comments on commit 50a145d

Please sign in to comment.