SAMZA-1198: disable the flaky test TestZkBarrierForVersionUpgrade.testZkBarrierForVersionUpgrade#116
SAMZA-1198: disable the flaky test TestZkBarrierForVersionUpgrade.testZkBarrierForVersionUpgrade#116fredji97 wants to merge 3 commits intoapache:masterfrom
Conversation
navina
left a comment
navina
left a comment
There was a problem hiding this comment.
@fredji97 Can you remove the entire stacktrace from the description of the PR and add a one line description of the problem?
the PR description becomes the summary of the github commit message and we don't want the entire stacktrace for it.
Also, please revise the title of the PR to be of the format "SAMZA- : <title>"
Thanks!
| zkServer.teardown(); | ||
| } | ||
|
|
||
| @Ignore("The test is flaky, see SAMZA-1193") |
There was a problem hiding this comment.
Can you please comment out the @test annotation instead of using Ignore annotation?
I believe last time @santhoshvenkat1988 tried with Ignore, it caused some failures in hudson. We decided that commenting out test annotation and adding a TODO with JIRA number was cleaner. You can talk to @santhoshvenkat1988 or @jmakes on why it failed on hudson.
fredji97
changed the title
fredji97
changed the title
2 participants
We are seeing the fails sometimes from this test, disabling it for build success first. See details in SAMZA-1198, and the JIRA for fix is in SAMZA-1193.