[STORM-3310] JCQueueTest is flaky

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.0.0
• Fix Version/s: 2.0.0
• Labels: pull-request-available
• Assignee: Stig Rohde Døssing
• Reporter: Stig Rohde Døssing
• Created: 06/Jan/19 22:20
• Updated: 16/Mar/19 03:26
• Resolved: 30/Jan/19 11:42

Description

The JCQueueTest is flaky

[ERROR] JCQueueTest.testFirstMessageFirst:61 We expect to receive first published message first, but received null expected:<FIRST> but was:<null>

The issue is that the test has a race condition. There is no check that the consumer thread has read all (or any) of the produced messages before the test terminates.

Issue Links

• links to: GitHub Pull Request #2934