mirror of https://github.com/apache/kafka.git
Fix bug in json naming (#18858)
Reviewers: Apoorv Mittal <apoorvmittal10@gmail.com>, Andrew Schofield <aschofield@confluent.io>
This commit is contained in:
parent
84b639d932
commit
210f277910
|
@ -98,7 +98,7 @@ public final class ShareConsumeBenchSpec extends TaskSpec {
|
||||||
@JsonProperty("bootstrapServers") String bootstrapServers,
|
@JsonProperty("bootstrapServers") String bootstrapServers,
|
||||||
@JsonProperty("targetMessagesPerSec") int targetMessagesPerSec,
|
@JsonProperty("targetMessagesPerSec") int targetMessagesPerSec,
|
||||||
@JsonProperty("maxMessages") long maxMessages,
|
@JsonProperty("maxMessages") long maxMessages,
|
||||||
@JsonProperty("consumerGroup") String shareGroup,
|
@JsonProperty("shareGroup") String shareGroup,
|
||||||
@JsonProperty("consumerConf") Map<String, String> consumerConf,
|
@JsonProperty("consumerConf") Map<String, String> consumerConf,
|
||||||
@JsonProperty("commonClientConf") Map<String, String> commonClientConf,
|
@JsonProperty("commonClientConf") Map<String, String> commonClientConf,
|
||||||
@JsonProperty("adminClientConf") Map<String, String> adminClientConf,
|
@JsonProperty("adminClientConf") Map<String, String> adminClientConf,
|
||||||
|
|
|
@ -50,6 +50,14 @@ class ShareConsumeBenchSpecTest {
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
public void testDefaultShareGroupName() {
|
||||||
|
ShareConsumeBenchSpec shareConsumeBenchSpec = new ShareConsumeBenchSpec(0, 0, "node", "localhost",
|
||||||
|
123, 1234, null, Collections.emptyMap(), Collections.emptyMap(), Collections.emptyMap(), 1,
|
||||||
|
Optional.empty(), List.of("abc"));
|
||||||
|
assertEquals("share", shareConsumeBenchSpec.shareGroup());
|
||||||
|
}
|
||||||
|
|
||||||
private ShareConsumeBenchSpec shareConsumeBenchSpec(List<String> activeTopics) {
|
private ShareConsumeBenchSpec shareConsumeBenchSpec(List<String> activeTopics) {
|
||||||
return new ShareConsumeBenchSpec(0, 0, "node", "localhost",
|
return new ShareConsumeBenchSpec(0, 0, "node", "localhost",
|
||||||
123, 1234, "sg-1",
|
123, 1234, "sg-1",
|
||||||
|
|
Loading…
Reference in New Issue