kafka-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ij...@apache.org
Subject kafka git commit: KAFKA-4526; Disable throttling test until it can be fixed correctly.
Date Mon, 19 Dec 2016 23:41:57 GMT
Repository: kafka
Updated Branches:
  refs/heads/trunk 529ca3293 -> 8dbdc90ba


KAFKA-4526; Disable throttling test until it can be fixed correctly.

At present, the test is fragile in the sense that the console consumer
has to start and be initialized before the verifiable producer begins
producing in the produce-consume-validate loop.

If this doesn't happen, the consumer will miss messages at the head of
the log and the test will fail.

At present, the consumer is considered inited once it has a PID. This is
a weak assumption. The plan is to poll appropriate metrics (like
partition assignment), and use those as a proxy for consumer
initialization. That work will be tracked in a separate ticket. For now,
we will disable the tests so that we can get the builds healthy again.

Author: Apurva Mehta <apurva.1618@gmail.com>

Reviewers: Ismael Juma <ismael@juma.me.uk>

Closes #2278 from apurvam/KAFKA-4526-throttling-test-failures


Project: http://git-wip-us.apache.org/repos/asf/kafka/repo
Commit: http://git-wip-us.apache.org/repos/asf/kafka/commit/8dbdc90b
Tree: http://git-wip-us.apache.org/repos/asf/kafka/tree/8dbdc90b
Diff: http://git-wip-us.apache.org/repos/asf/kafka/diff/8dbdc90b

Branch: refs/heads/trunk
Commit: 8dbdc90bae940b47dfaf28a0066b061ebfa87b87
Parents: 529ca32
Author: Apurva Mehta <apurva.1618@gmail.com>
Authored: Mon Dec 19 23:41:56 2016 +0000
Committer: Ismael Juma <ismael@juma.me.uk>
Committed: Mon Dec 19 23:41:56 2016 +0000

----------------------------------------------------------------------
 tests/kafkatest/tests/core/throttling_test.py | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/kafka/blob/8dbdc90b/tests/kafkatest/tests/core/throttling_test.py
----------------------------------------------------------------------
diff --git a/tests/kafkatest/tests/core/throttling_test.py b/tests/kafkatest/tests/core/throttling_test.py
index 9684099..80b5658 100644
--- a/tests/kafkatest/tests/core/throttling_test.py
+++ b/tests/kafkatest/tests/core/throttling_test.py
@@ -15,7 +15,7 @@
 
 import time
 import math
-from ducktape.mark import parametrize
+from ducktape.mark import parametrize,ignore
 from ducktape.mark.resource import cluster
 from ducktape.utils.util import wait_until
 
@@ -138,6 +138,7 @@ class ThrottlingTest(ProduceConsumeValidateTest):
                 estimated_throttled_time,
                 time_taken))
 
+    @ignore
     @cluster(num_nodes=10)
     @parametrize(bounce_brokers=False)
     @parametrize(bounce_brokers=True)


Mime
View raw message