Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Flaky-test: StreamingEntryReaderTests.testCanReadEntryFromMLedgerSizeExceededLimit #9885

Closed
lhotari opened this issue Mar 11, 2021 · 1 comment · Fixed by #9905
Closed

Flaky-test: StreamingEntryReaderTests.testCanReadEntryFromMLedgerSizeExceededLimit #9885

lhotari opened this issue Mar 11, 2021 · 1 comment · Fixed by #9905

Comments

@lhotari
Copy link
Member

lhotari commented Mar 11, 2021

StreamingEntryReaderTests is flaky. The testCanReadEntryFromMLedgerSizeExceededLimit test method fails sporadically.

It seems that the 300 millisecond timeout for async operations is too short.

example failure

org.awaitility.core.ConditionTimeoutException: Condition with lambda expression in
 org.apache.pulsar.broker.service.streamingdispatch.StreamingEntryReaderTests that uses 
java.util.concurrent.atomic.AtomicBoolean was not fulfilled within 300 milliseconds.
 	at org.awaitility.core.ConditionAwaiter.await(ConditionAwaiter.java:165)
 	at org.awaitility.core.CallableCondition.await(CallableCondition.java:78)
 	at org.awaitility.core.CallableCondition.await(CallableCondition.java:26)
 	at org.awaitility.core.ConditionFactory.until(ConditionFactory.java:895)
 	at org.awaitility.core.ConditionFactory.until(ConditionFactory.java:864)
 	at org.apache.pulsar.broker.service.streamingdispatch.StreamingEntryReaderTests.testCanReadEntryFromMLedgerSizeExceededLimit(StreamingEntryReaderTests.java:182)
Full exception stacktrace
 [INFO] Running org.apache.pulsar.broker.service.streamingdispatch.StreamingEntryReaderTests
 [ERROR] Tests run: 7, Failures: 1, Errors: 0, Skipped: 1, Time elapsed: 3.543 s <<< FAILURE! - in org.apache.pulsar.broker.service.streamingdispatch.StreamingEntryReaderTests
 [ERROR] testCanReadEntryFromMLedgerSizeExceededLimit(org.apache.pulsar.broker.service.streamingdispatch.StreamingEntryReaderTests)  Time elapsed: 0.346 s  <<< FAILURE!
 org.awaitility.core.ConditionTimeoutException: Condition with lambda expression in org.apache.pulsar.broker.service.streamingdispatch.StreamingEntryReaderTests that uses java.util.concurrent.atomic.AtomicBoolean was not fulfilled within 300 milliseconds.
 	at org.awaitility.core.ConditionAwaiter.await(ConditionAwaiter.java:165)
 	at org.awaitility.core.CallableCondition.await(CallableCondition.java:78)
 	at org.awaitility.core.CallableCondition.await(CallableCondition.java:26)
 	at org.awaitility.core.ConditionFactory.until(ConditionFactory.java:895)
 	at org.awaitility.core.ConditionFactory.until(ConditionFactory.java:864)
 	at org.apache.pulsar.broker.service.streamingdispatch.StreamingEntryReaderTests.testCanReadEntryFromMLedgerSizeExceededLimit(StreamingEntryReaderTests.java:182)
 	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
 	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
 	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
 	at java.lang.reflect.Method.invoke(Method.java:498)
 	at org.testng.internal.MethodInvocationHelper.invokeMethod(MethodInvocationHelper.java:132)
 	at org.testng.internal.InvokeMethodRunnable.runOne(InvokeMethodRunnable.java:45)
 	at org.testng.internal.InvokeMethodRunnable.call(InvokeMethodRunnable.java:73)
 	at org.testng.internal.InvokeMethodRunnable.call(InvokeMethodRunnable.java:11)
 	at java.util.concurrent.FutureTask.run(FutureTask.java:266)
 	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
 	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
 	at java.lang.Thread.run(Thread.java:748)
@lhotari
Copy link
Member Author

lhotari commented Mar 11, 2021

@MarvinCai Would you mind taking a look? This test was introduced in PR #9056 .

sijie pushed a commit that referenced this issue Mar 16, 2021
…der to reduce flakiness. (#9905)

Fixes #9885

### Motivation
StreamingEntryReaderTests is flaky. The testCanReadEntryFromMLedgerSizeExceededLimit test method fails sporadically.

### Modifications
Increased timeout to ease the bound.
fmiguelez pushed a commit to fmiguelez/pulsar that referenced this issue Mar 16, 2021
…der to reduce flakiness. (apache#9905)

Fixes apache#9885

### Motivation
StreamingEntryReaderTests is flaky. The testCanReadEntryFromMLedgerSizeExceededLimit test method fails sporadically.

### Modifications
Increased timeout to ease the bound.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant