Sending an exchange in a thread created by the bean result in a NullpointerException on DeliveryChannel.send()

• Key: SMXCOMP-335
• Type: Bug
• Status: Resolved
• Resolution: Fixed
• Priority: Blocker
• Assignee: Guillaume Nodet
• Reporter: Ryan Bohn
• Component/s: servicemix-bean
• Fix Version/s: 3.2.4, servicemix-bean-2008.01

Description

I'm trying to send a message asynchronously from a sender to a receiver using an InOnly exchange. When I call DeliveryChannel.send(), I get the following NullPointerException:

Sending message: Hello?
java.lang.NullPointerException
at java.util.concurrent.ConcurrentHashMap.put(ConcurrentHashMap.java:846)
at org.apache.servicemix.bean.BeanEndpoint$PojoChannel.send(BeanEndpoint.java:566)
at example.Sender$1.run(Sender.java:40)
at java.lang.Thread.run(Thread.java:613)

When I go to the line of code in the BeanEndpoint, I see the following:

requests.put(messageExchange.getExchangeId(), currentRequest.get());

Using my debugger, I noticed the value of currentRequest.get() returns null, which is not allowed in a ConcurrentHashMap (neither null keys nor null values are allowed). the message exchange id is non-null.

I also noticed that currentRequest is only set when the component receives an exchange, which is not the case here.

Both the sender and receiver are using bean endpoints. I have attached the project to this ticket. You can deploy the project using maven, with the following command:

mvn install jbi:projectDeploy

Comments

[Ryan Bohn]

any word on this?

[Guillaume Nodet]

This is a use case that is not really handled at the moment.
It would require some modifications in the component to handle that.
As a workaround, using sendSync should work.

[Jeroen Ninck Blok]

I have attached a patch to this issue. I am pretty sure that it doesn't fix the problem as Guillaume describes in his comment, however it does appear to be working without causing any (serious) memory leaks.

[Guillaume Nodet]

Thanks a lot for this patch.

Sending src/main/java/org/apache/servicemix/bean/BeanEndpoint.java
Sending src/test/java/org/apache/servicemix/bean/ConsumerBeanTest.java
Adding src/test/java/org/apache/servicemix/bean/SenderBeanTest.java
Adding src/test/java/org/apache/servicemix/bean/beans/SenderBean.java
Adding src/test/resources/sender-bean.xml
Transmitting file data .....
Committed revision 691553.

[Guillaume Nodet]

Sending src/main/java/org/apache/servicemix/bean/BeanEndpoint.java
Transmitting file data .
Committed revision 713346.

Backported to 3.2 branch