Skip to content

Commit

Permalink
KAFKA-18025 Remove flaky tag from several tests in ShareConsumerTest (#…
Browse files Browse the repository at this point in the history
  • Loading branch information
ShivsundarR authored Dec 17, 2024
1 parent 337fb8c commit e88fede
Showing 1 changed file with 0 additions and 7 deletions.
7 changes: 0 additions & 7 deletions core/src/test/java/kafka/test/api/ShareConsumerTest.java
Original file line number Diff line number Diff line change
Expand Up @@ -275,7 +275,6 @@ public void testSubscriptionAndPollMultiple(String persister) {
}
}

@Flaky("KAFKA-18033")
@ParameterizedTest(name = "{displayName}.persister={0}")
@ValueSource(strings = {NO_OP_PERSISTER, DEFAULT_STATE_PERSISTER})
public void testAcknowledgementSentOnSubscriptionChange(String persister) throws ExecutionException, InterruptedException {
Expand Down Expand Up @@ -599,7 +598,6 @@ public void testExplicitAcknowledgeCommitSuccess(String persister) {
}
}

@Flaky("KAFKA-18033")
@ParameterizedTest(name = "{displayName}.persister={0}")
@ValueSource(strings = {NO_OP_PERSISTER, DEFAULT_STATE_PERSISTER})
public void testExplicitAcknowledgementCommitAsync(String persister) throws InterruptedException {
Expand Down Expand Up @@ -656,7 +654,6 @@ public void testExplicitAcknowledgementCommitAsync(String persister) throws Inte
}
}

@Flaky("KAFKA-18033")
@ParameterizedTest(name = "{displayName}.persister={0}")
@ValueSource(strings = {NO_OP_PERSISTER, DEFAULT_STATE_PERSISTER})
public void testExplicitAcknowledgementCommitAsyncPartialBatch(String persister) {
Expand Down Expand Up @@ -763,7 +760,6 @@ public void testExplicitAcknowledgeReleaseAccept(String persister) {
}
}

@Flaky("KAFKA-18033")
@ParameterizedTest(name = "{displayName}.persister={0}")
@ValueSource(strings = {NO_OP_PERSISTER, DEFAULT_STATE_PERSISTER})
public void testExplicitAcknowledgeReleaseClose(String persister) {
Expand Down Expand Up @@ -1188,7 +1184,6 @@ public void testMultipleConsumersInGroupFailureConcurrentConsumption(String pers
assertEquals(producerCount * messagesPerProducer, totalSuccessResult);
}

@Flaky("KAFKA-18025")
@ParameterizedTest(name = "{displayName}.persister={0}")
@ValueSource(strings = {NO_OP_PERSISTER, DEFAULT_STATE_PERSISTER})
public void testAcquisitionLockTimeoutOnConsumer(String persister) throws InterruptedException {
Expand Down Expand Up @@ -1296,7 +1291,6 @@ public void onComplete(Map<TopicIdPartition, Set<Long>> offsetsMap, Exception ex
* Test to verify that the acknowledgement commit callback can invoke KafkaShareConsumer.wakeup() and it
* wakes up the enclosing poll.
*/
@Flaky("KAFKA-18033")
@ParameterizedTest(name = "{displayName}.persister={0}")
@ValueSource(strings = {NO_OP_PERSISTER, DEFAULT_STATE_PERSISTER})
public void testAcknowledgementCommitCallbackCallsShareConsumerWakeup(String persister) throws InterruptedException {
Expand Down Expand Up @@ -1349,7 +1343,6 @@ public void onComplete(Map<TopicIdPartition, Set<Long>> offsetsMap, Exception ex
* Test to verify that the acknowledgement commit callback can throw an exception, and it is propagated
* to the caller of poll().
*/
@Flaky("KAFKA-18033")
@ParameterizedTest(name = "{displayName}.persister={0}")
@ValueSource(strings = {NO_OP_PERSISTER, DEFAULT_STATE_PERSISTER})
public void testAcknowledgementCommitCallbackThrowsException(String persister) throws InterruptedException {
Expand Down

0 comments on commit e88fede

Please sign in to comment.