[CAMEL-8366] Close used iterator if RuntimeException is thrown at next()/hasNext() in Splitter

• Type: Improvement
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 2.14.1
• Fix Version/s: 2.14.2, 2.15.0
• Component/s: camel-core
• Estimated Complexity: Unknown
• Assignee: Claus Ibsen
• Reporter: Tomohisa Igarashi
• Created: 17/Feb/15 09:46
• Updated: 18/Feb/15 07:30
• Resolved: 18/Feb/15 07:19

Description

There're some spots using user provided iterator but don't close them if RuntimeException is thrown during iteration even if it's Closeable . It would be convenient if it is closed in finally block.

I immediately found 2, guessing there're some more else:
https://github.com/apache/camel/blob/master/camel-core/src/main/java/org/apache/camel/processor/Splitter.java#L232-L234
https://github.com/apache/camel/blob/master/camel-core/src/main/java/org/apache/camel/processor/MulticastProcessor.java#L267-L347