chia7712 merged PR #16638:
URL: https://github.com/apache/kafka/pull/16638
--
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.
To unsubscribe, e-mail: jira-unsubscr...@kafka.apache
TaiJuWu commented on PR #16638:
URL: https://github.com/apache/kafka/pull/16638#issuecomment-2246655210
> @TaiJuWu please use the new helper to rewrite
>
>
https://github.com/apache/kafka/blob/c71eb60a3b2e30cc376b81144cf9c2c411d8f28b/clients/src/test/java/org/apache/kafka/clients/cons
chia7712 commented on PR #16638:
URL: https://github.com/apache/kafka/pull/16638#issuecomment-2245651512
@TaiJuWu please use the new helper to rewrite
https://github.com/apache/kafka/blob/c71eb60a3b2e30cc376b81144cf9c2c411d8f28b/clients/src/test/java/org/apache/kafka/clients/consumer/KafkaCo
TaiJuWu commented on code in PR #16638:
URL: https://github.com/apache/kafka/pull/16638#discussion_r1687898103
##
clients/src/test/java/org/apache/kafka/clients/consumer/KafkaConsumerTest.java:
##
@@ -2965,7 +2970,28 @@ public void testSubscriptionOnInvalidTopic(GroupProtocol
g
TaiJuWu commented on code in PR #16638:
URL: https://github.com/apache/kafka/pull/16638#discussion_r1687892353
##
clients/src/test/java/org/apache/kafka/clients/consumer/KafkaConsumerTest.java:
##
@@ -2965,7 +2976,21 @@ public void testSubscriptionOnInvalidTopic(GroupProtocol
g
lianetm commented on code in PR #16638:
URL: https://github.com/apache/kafka/pull/16638#discussion_r1686689029
##
clients/src/test/java/org/apache/kafka/clients/consumer/KafkaConsumerTest.java:
##
@@ -2093,14 +2093,27 @@ public void
testEndOffsetsAuthenticationFailure(GroupProt
lianetm commented on code in PR #16638:
URL: https://github.com/apache/kafka/pull/16638#discussion_r1686689029
##
clients/src/test/java/org/apache/kafka/clients/consumer/KafkaConsumerTest.java:
##
@@ -2093,14 +2093,27 @@ public void
testEndOffsetsAuthenticationFailure(GroupProt
lianetm commented on code in PR #16638:
URL: https://github.com/apache/kafka/pull/16638#discussion_r1686692620
##
clients/src/test/java/org/apache/kafka/clients/consumer/KafkaConsumerTest.java:
##
@@ -2965,7 +2976,21 @@ public void testSubscriptionOnInvalidTopic(GroupProtocol
g
lianetm commented on code in PR #16638:
URL: https://github.com/apache/kafka/pull/16638#discussion_r1686693862
##
clients/src/test/java/org/apache/kafka/clients/consumer/KafkaConsumerTest.java:
##
@@ -3399,4 +3424,11 @@ public void configure(Map configs) {
CLIENT_I
lianetm commented on code in PR #16638:
URL: https://github.com/apache/kafka/pull/16638#discussion_r1686692620
##
clients/src/test/java/org/apache/kafka/clients/consumer/KafkaConsumerTest.java:
##
@@ -2965,7 +2976,21 @@ public void testSubscriptionOnInvalidTopic(GroupProtocol
g
lianetm commented on code in PR #16638:
URL: https://github.com/apache/kafka/pull/16638#discussion_r1686689029
##
clients/src/test/java/org/apache/kafka/clients/consumer/KafkaConsumerTest.java:
##
@@ -2093,14 +2093,27 @@ public void
testEndOffsetsAuthenticationFailure(GroupProt
lianetm commented on code in PR #16638:
URL: https://github.com/apache/kafka/pull/16638#discussion_r1686665604
##
clients/src/test/java/org/apache/kafka/clients/consumer/KafkaConsumerTest.java:
##
@@ -862,7 +862,7 @@ public void
testFetchProgressWithMissingPartitionPosition(Gro
chia7712 commented on code in PR #16638:
URL: https://github.com/apache/kafka/pull/16638#discussion_r1685701571
##
clients/src/test/java/org/apache/kafka/clients/consumer/KafkaConsumerTest.java:
##
@@ -2965,7 +2976,21 @@ public void testSubscriptionOnInvalidTopic(GroupProtocol
TaiJuWu commented on code in PR #16638:
URL: https://github.com/apache/kafka/pull/16638#discussion_r1685589970
##
clients/src/test/java/org/apache/kafka/clients/consumer/KafkaConsumerTest.java:
##
@@ -2965,7 +2976,21 @@ public void testSubscriptionOnInvalidTopic(GroupProtocol
g
chia7712 commented on code in PR #16638:
URL: https://github.com/apache/kafka/pull/16638#discussion_r1685477697
##
clients/src/test/java/org/apache/kafka/clients/consumer/KafkaConsumerTest.java:
##
@@ -2965,7 +2976,21 @@ public void testSubscriptionOnInvalidTopic(GroupProtocol
TaiJuWu opened a new pull request, #16638:
URL: https://github.com/apache/kafka/pull/16638
*More detailed description of your change,
See [KAFKA-17163](https://issues.apache.org/jira/browse/KAFKA-17163)
*Summary of testing strategy (including rationale)
### Committer Checklis
16 matches
Mail list logo