Re: [PR] KAFKA-15756: Migrate existing integration tests to run old protocol in new coordinator [kafka]

2023-11-14 Thread via GitHub
dongnuo123 commented on code in PR #14675: URL: https://github.com/apache/kafka/pull/14675#discussion_r1393317417 ## core/src/test/scala/integration/kafka/api/PlaintextConsumerTest.scala: ## @@ -547,7 +565,7 @@ class PlaintextConsumerTest extends BaseConsumerTest {

Re: [PR] KAFKA-15756: Migrate existing integration tests to run old protocol in new coordinator [kafka]

2023-11-14 Thread via GitHub
dongnuo123 commented on code in PR #14675: URL: https://github.com/apache/kafka/pull/14675#discussion_r1393317417 ## core/src/test/scala/integration/kafka/api/PlaintextConsumerTest.scala: ## @@ -547,7 +565,7 @@ class PlaintextConsumerTest extends BaseConsumerTest {

Re: [PR] KAFKA-15756: Migrate existing integration tests to run old protocol in new coordinator [kafka]

2023-11-13 Thread via GitHub
dongnuo123 commented on code in PR #14675: URL: https://github.com/apache/kafka/pull/14675#discussion_r1391629976 ## core/src/test/scala/unit/kafka/admin/ListConsumerGroupTest.scala: ## @@ -29,7 +29,7 @@ import org.junit.jupiter.params.provider.ValueSource class

Re: [PR] KAFKA-15756: Migrate existing integration tests to run old protocol in new coordinator [kafka]

2023-11-06 Thread via GitHub
dongnuo123 commented on code in PR #14675: URL: https://github.com/apache/kafka/pull/14675#discussion_r1384081702 ## core/src/test/scala/integration/kafka/api/SaslMultiMechanismConsumerTest.scala: ## @@ -41,8 +43,9 @@ class SaslMultiMechanismConsumerTest extends

Re: [PR] KAFKA-15756: Migrate existing integration tests to run old protocol in new coordinator [kafka]

2023-11-06 Thread via GitHub
dongnuo123 commented on code in PR #14675: URL: https://github.com/apache/kafka/pull/14675#discussion_r1383950478 ## core/src/test/scala/integration/kafka/api/PlaintextConsumerTest.scala: ## @@ -547,7 +565,7 @@ class PlaintextConsumerTest extends BaseConsumerTest {

Re: [PR] KAFKA-15756: Migrate existing integration tests to run old protocol in new coordinator [kafka]

2023-11-06 Thread via GitHub
dongnuo123 commented on code in PR #14675: URL: https://github.com/apache/kafka/pull/14675#discussion_r1383921029 ## core/src/test/scala/integration/kafka/api/PlaintextConsumerTest.scala: ## @@ -1805,16 +1865,19 @@ class PlaintextConsumerTest extends BaseConsumerTest {

Re: [PR] KAFKA-15756: Migrate existing integration tests to run old protocol in new coordinator [kafka]

2023-11-03 Thread via GitHub
dongnuo123 commented on code in PR #14675: URL: https://github.com/apache/kafka/pull/14675#discussion_r1382138739 ## core/src/test/scala/integration/kafka/api/ConsumerBounceTest.scala: ## @@ -464,7 +477,7 @@ class ConsumerBounceTest extends AbstractConsumerTest with Logging {

Re: [PR] KAFKA-15756: Migrate existing integration tests to run old protocol in new coordinator [kafka]

2023-11-03 Thread via GitHub
dongnuo123 commented on code in PR #14675: URL: https://github.com/apache/kafka/pull/14675#discussion_r1382143423 ## core/src/test/scala/integration/kafka/api/ConsumerBounceTest.scala: ## @@ -340,8 +351,9 @@ class ConsumerBounceTest extends AbstractConsumerTest with Logging {

Re: [PR] KAFKA-15756: Migrate existing integration tests to run old protocol in new coordinator [kafka]

2023-11-03 Thread via GitHub
dongnuo123 commented on code in PR #14675: URL: https://github.com/apache/kafka/pull/14675#discussion_r1382141336 ## core/src/test/scala/integration/kafka/api/ConsumerBounceTest.scala: ## @@ -77,8 +83,9 @@ class ConsumerBounceTest extends AbstractConsumerTest with Logging {

Re: [PR] KAFKA-15756: Migrate existing integration tests to run old protocol in new coordinator [kafka]

2023-11-03 Thread via GitHub
dongnuo123 commented on code in PR #14675: URL: https://github.com/apache/kafka/pull/14675#discussion_r1382138739 ## core/src/test/scala/integration/kafka/api/ConsumerBounceTest.scala: ## @@ -464,7 +477,7 @@ class ConsumerBounceTest extends AbstractConsumerTest with Logging {

[PR] KAFKA-15756: Migrate existing integration tests to run old protocol in new coordinator [kafka]

2023-10-31 Thread via GitHub
dongnuo123 opened a new pull request, #14675: URL: https://github.com/apache/kafka/pull/14675 The list of the existing integration tests to migrate are as follows. - QuorumTestHarness - ReassignPartitionsIntegrationTest - EpochDrivenReplicationProtocolAcceptanceTest -