You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kafka.apache.org by bb...@apache.org on 2019/09/04 15:09:24 UTC
[kafka] 01/02: KAFKA-8861 Fix flaky
RegexSourceIntegrationTest.testMultipleConsumersCanReadFromPartitionedTopic
(#7281)
This is an automated email from the ASF dual-hosted git repository.
bbejeck pushed a commit to branch 2.2
in repository https://gitbox.apache.org/repos/asf/kafka.git
commit aa0c76af8e02eaa6112b2ff94beaf4e7b9629449
Author: Chia-Ping Tsai <ch...@gmail.com>
AuthorDate: Wed Sep 4 22:13:56 2019 +0800
KAFKA-8861 Fix flaky RegexSourceIntegrationTest.testMultipleConsumersCanReadFromPartitionedTopic (#7281)
similar to https://issues.apache.org/jira/browse/KAFKA-8011 and https://issues.apache.org/jira/browse/KAFKA-8026
Reviewers: Matthias J. Sax <mj...@apache.org>, Bill Bejeck <bb...@gmail.com>
---
.../apache/kafka/streams/integration/RegexSourceIntegrationTest.java | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/streams/src/test/java/org/apache/kafka/streams/integration/RegexSourceIntegrationTest.java b/streams/src/test/java/org/apache/kafka/streams/integration/RegexSourceIntegrationTest.java
index fc7a14c..26543d1 100644
--- a/streams/src/test/java/org/apache/kafka/streams/integration/RegexSourceIntegrationTest.java
+++ b/streams/src/test/java/org/apache/kafka/streams/integration/RegexSourceIntegrationTest.java
@@ -312,8 +312,8 @@ public class RegexSourceIntegrationTest {
partitionedStreamLeader.to(DEFAULT_OUTPUT_TOPIC, Produced.with(stringSerde, stringSerde));
partitionedStreamFollower.to(DEFAULT_OUTPUT_TOPIC, Produced.with(stringSerde, stringSerde));
- final List<String> leaderAssignment = new ArrayList<>();
- final List<String> followerAssignment = new ArrayList<>();
+ final List<String> leaderAssignment = new CopyOnWriteArrayList<>();
+ final List<String> followerAssignment = new CopyOnWriteArrayList<>();
partitionedStreamsLeader = new KafkaStreams(builderLeader.build(), streamsConfiguration, new DefaultKafkaClientSupplier() {
@Override