[SMX4-516] The camel-nmr ServiceMixProducer creates a new Channel per-exchange, also doesn't close it.

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 4.0.0
• Fix Version/s: 4.2.0
• Patch Info: Patch Available
• Assignee: Freeman Yue Fang
• Reporter: Stanley J. Lewis
• Created: 12/Apr/10 14:05
• Updated: 13/Apr/10 06:30
• Resolved: 13/Apr/10 06:30

Description

In ServiceMixProducer.java we create a Channel for each exchange passed into process(). Under the covers this will cause a new thread pool to be created, as when you create a Channel that creates a new Executor instance, have a look in ServiceMixImpl.createChannel() in the NMR.
Instead this component should just create the Channel once and re-use it for subsequent exchanges.

Also ServiceMixProducer doesn't properly close the Channel it's creating which may lead to leaking resources.