You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kafka.apache.org by rh...@apache.org on 2020/02/05 15:48:08 UTC

[kafka] branch 2.3 updated: KAFKA-7489: Backport removal of 0.9 compatibility checks from ConnectDistributedTest (#8035)

This is an automated email from the ASF dual-hosted git repository.

rhauch pushed a commit to branch 2.3
in repository https://gitbox.apache.org/repos/asf/kafka.git


The following commit(s) were added to refs/heads/2.3 by this push:
     new 8a4d9f5  KAFKA-7489: Backport removal of 0.9 compatibility checks from ConnectDistributedTest (#8035)
8a4d9f5 is described below

commit 8a4d9f524fa7cc69f66b883fee52d5e5bc5eee07
Author: Greg Harris <gr...@confluent.io>
AuthorDate: Wed Feb 5 07:47:29 2020 -0800

    KAFKA-7489: Backport removal of 0.9 compatibility checks from ConnectDistributedTest (#8035)
    
    (#7023) exposed an incompatibility between Kafka <=0.9 and Connect >0.9,
    in which the broker does not recognize a request for ApiVersions.
    For trunk and 2.4, this test case was removed rather than the issue addressed.
    This effectively backports the other half of (#7023) which was left out of (#7791).
    
    Signed-off-by: Greg Harris <gr...@confluent.io>
    
    Author: Greg Harris <gr...@confluent.io>
    Reviewers: Randall Hauch <rh...@gmail.com>, Andrew Choi <an...@linkedin.com>
---
 tests/kafkatest/tests/connect/connect_distributed_test.py | 2 --
 1 file changed, 2 deletions(-)

diff --git a/tests/kafkatest/tests/connect/connect_distributed_test.py b/tests/kafkatest/tests/connect/connect_distributed_test.py
index 0dddeee..830fbce 100644
--- a/tests/kafkatest/tests/connect/connect_distributed_test.py
+++ b/tests/kafkatest/tests/connect/connect_distributed_test.py
@@ -528,7 +528,6 @@ class ConnectDistributedTest(Test):
     @parametrize(broker_version=str(LATEST_0_10_2), auto_create_topics=False, security_protocol=SecurityConfig.PLAINTEXT, connect_protocol='compatible')
     @parametrize(broker_version=str(LATEST_0_10_1), auto_create_topics=False, security_protocol=SecurityConfig.PLAINTEXT, connect_protocol='compatible')
     @parametrize(broker_version=str(LATEST_0_10_0), auto_create_topics=True, security_protocol=SecurityConfig.PLAINTEXT, connect_protocol='compatible')
-    @parametrize(broker_version=str(LATEST_0_9), auto_create_topics=True, security_protocol=SecurityConfig.PLAINTEXT, connect_protocol='compatible')
     @parametrize(broker_version=str(DEV_BRANCH), auto_create_topics=False, security_protocol=SecurityConfig.PLAINTEXT, connect_protocol='eager')
     @parametrize(broker_version=str(LATEST_2_3), auto_create_topics=False, security_protocol=SecurityConfig.PLAINTEXT, connect_protocol='eager')
     @parametrize(broker_version=str(LATEST_2_2), auto_create_topics=False, security_protocol=SecurityConfig.PLAINTEXT, connect_protocol='eager')
@@ -540,7 +539,6 @@ class ConnectDistributedTest(Test):
     @parametrize(broker_version=str(LATEST_0_10_2), auto_create_topics=False, security_protocol=SecurityConfig.PLAINTEXT, connect_protocol='eager')
     @parametrize(broker_version=str(LATEST_0_10_1), auto_create_topics=False, security_protocol=SecurityConfig.PLAINTEXT, connect_protocol='eager')
     @parametrize(broker_version=str(LATEST_0_10_0), auto_create_topics=True, security_protocol=SecurityConfig.PLAINTEXT, connect_protocol='eager')
-    @parametrize(broker_version=str(LATEST_0_9), auto_create_topics=True, security_protocol=SecurityConfig.PLAINTEXT, connect_protocol='eager')
     def test_broker_compatibility(self, broker_version, auto_create_topics, security_protocol, connect_protocol):
         """
         Verify that Connect will start up with various broker versions with various configurations.