[CAMEL-5730] SqsConsumer receive thread is tightly coupled to extendMessageVisiblity's scheduled task

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.10.0
• Fix Version/s: 2.10.3, 2.11.0
• Component/s: camel-aws
• Patch Info: Patch Available
• Estimated Complexity: Novice
• Assignee: Claus Ibsen
• Reporter: alex hutter
• Created: 22/Oct/12 13:13
• Updated: 22/Oct/12 13:37
• Resolved: 22/Oct/12 13:37

Description

When I made the patch file for CAMEL-5306 I made a mistake and coupled the cancelling of the visibility extending scheduled task with the thread that receives the message inside SqsConsumer. This causes issues when try to write Processors that delegate the completion of an exchange to another thread.

Actually the cancelling of the scheduled task should have been handled via exchange.addOnCompletion(), the same way the the overall SQS transaction is completed.

The attached patch fixes this issue.