This is an automated email from the ASF dual-hosted git repository.
ableegoldman pushed a commit to branch trunk
in repository https://gitbox.apache.org/repos/asf/kafka.git
The following commit(s) were added to refs/heads/trunk by this push:
new 3799708ff09 KAFKA-14533: re-enable 'false' and disable the 'true'
parameter of SmokeTestDriverIntegrationTest (#13156)
3799708ff09 is described below
commit 3799708ff09b174883eddc1f6a857ddc518fa35f
Author: A. Sophie Blee-Goldman <[email protected]>
AuthorDate: Tue Jan 24 00:14:20 2023 -0800
KAFKA-14533: re-enable 'false' and disable the 'true' parameter of
SmokeTestDriverIntegrationTest (#13156)
I immediately saw a failure with stateUpdaterEnabled = true after disabling
the false parameter, which suggests the problem actually does lie in the state
updater itself and not the act of parametrization of the test. To verify this
theory, and help stabilize the 3.4 release branch, let's try one more test by
swapping out the true build in favor of the false one. If the listOffsets
requests stop failing and causing this integration test to hit the global
timeout as is currently happen [...]
After getting in a few builds to see whether the flakiness subsides, we
should merge this PR to re-enable both parameters going forward:
https://github.com/apache/kafka/pull/13155
Reviewers: Anna Sophie Blee-Goldman <[email protected]>
---
.../kafka/streams/integration/SmokeTestDriverIntegrationTest.java | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git
a/streams/src/test/java/org/apache/kafka/streams/integration/SmokeTestDriverIntegrationTest.java
b/streams/src/test/java/org/apache/kafka/streams/integration/SmokeTestDriverIntegrationTest.java
index 34336c05046..cbe3afc7bae 100644
---
a/streams/src/test/java/org/apache/kafka/streams/integration/SmokeTestDriverIntegrationTest.java
+++
b/streams/src/test/java/org/apache/kafka/streams/integration/SmokeTestDriverIntegrationTest.java
@@ -97,8 +97,8 @@ public class SmokeTestDriverIntegrationTest {
private static Stream<Boolean> parameters() {
return Stream.of(
// TODO KAFKA-14533: debug and re-enable both parameters
- Boolean.TRUE
- //Boolean.FALSE
+ //Boolean.TRUE
+ Boolean.FALSE
);
}