[CAMEL-3657] Subtle test fails due multiple threads involved and visibility issues in mock endpoints

• Type: Task
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.6.0
• Fix Version/s: 2.7.0
• Component/s: camel-core, camel-hawtdb
• Assignee: Claus Ibsen
• Reporter: Claus Ibsen
• Created: 11/Feb/11 12:12
• Updated: 25/Oct/11 11:36
• Resolved: 11/Feb/11 12:32

Description

The test HawtDBAggregateRecoverDeadLetterChannelFailedTest may fail with a header missing. This seems really odd. And tracking this down to a thread visibility issue with the thread performing the assertion on the mock, and threads routing the messages in Camel. State on the mock should be volatile so changes is visible between the threads.

Also optimize getHeader on Message to check for any headers first, avoiding the lazy create a Map, when really not needed.