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
[fix][flaky-test]ManagedCursorMetricsTest.testCursorReadWriteMetrics #17045
[fix][flaky-test]ManagedCursorMetricsTest.testCursorReadWriteMetrics #17045
Conversation
return managedCursor.getStats(); | ||
} | ||
|
||
@Test(timeOut = 2 * 1000) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We can just use the default timeout? Or any reason to set it to 2 seconds for this test? It might become flaky in the CI environment.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Or any reason to set it to 2 seconds for this test?
no
We can just use the default timeout?
already change to default timeout.
/pulsarbot rerun-failure-checks |
2 similar comments
/pulsarbot rerun-failure-checks |
/pulsarbot rerun-failure-checks |
…pache#17045) (cherry picked from commit e0ff3d7) (cherry picked from commit eece41b)
Fixes #16948
Master Issue: #16948
Motivation
This test is flaky because that BK has not been successfully written when the asset executed
pulsar/pulsar-broker/src/test/java/org/apache/pulsar/broker/stats/ManagedCursorMetricsTest.java
Lines 244 to 248 in e5d5fbe
Modifications
Makes assert execute after BK written
Documentation
doc-required
(Your PR needs to update docs and you will update later)
doc-not-needed
(Please explain why)
doc
(Your PR contains doc changes)
doc-complete
(Docs have been already added)