JMS Request/Response example

Question

Hi
I am trying to implement a JMS Request/Response example on glassfish, but i am not getting the correct behaviour.
My code is below. I am sending a message to a queue and adding the setJMSReplyTo another queue. I call the recv.receive(10000); and wait for the messages to be received. But this call blocks the current thread and the MDB that i orginally sent the message to only gets executed after the recv.receive(10000); has timed out after 10 seconds.
Can someone confirm that my code is correct or am i doing something wrong?
Connection connection = null;
Session session = null;
String text = "hello";
try {
System.out.println("Sending " + text);
connection = searchDestFactory.createConnection();
session = connection.createSession(false, Session.AUTO_ACKNOWLEDGE);
MessageProducer messageProducer = session.createProducer(searchDest);
TextMessage tm = session.createTextMessage();
tm.setText(text);
tm.setJMSReplyTo(destQueue);
messageProducer.send(tm);
System.out.println("Sent " + text);
MessageConsumer recv = session.createConsumer(destQueue);
connection.start();
Message m = recv.receive(10000);
tm = (TextMessage) m;
if(tm != null) System.out.println(tm.getText());
else System.out.println("No message replied");
} catch (JMSException ex) {
System.out.println(ex);
Thanks Glen
Edited by: glen_ on Jun 16, 2008 6:13 AM
Edited by: glen_ on Jun 16, 2008 6:13 AM
Edited by: glen_ on Jun 16, 2008 6:14 AM

Answer 1

Glen,
I have never attempted to use the messaging service the way you have, namely a single instance as both sender and receiver, but I noticed that you do send the message before you register your Consumer. My first and easiest suggestion would be to simply move your consumer block (I would move both lines) above the producer block and try again.
If that attempt fails, I would implement a MessageListener, once again before the producer block and allow it to handle received messages (no need for recv.receive(10000);)
Example:
    public class QueueMessageListener implements MessageListener {
        public void onMessage(Message message) {
            try {
                System.out.println(String.format("From Glassfish: %s received a %s of type %s.", m_Queue.getQueueName(), message.getClass().getName(), message.getJMSType()));
                System.out.println(printJMSMessage(message));
            } catch (JMSException ex) {
            //handle message here
    }and somewhere before the producer block:
            m_msgListener = new QueueMessageListener();
            m_msgConsumer =  m_Session.createConsumer(m_Queue);
            m_msgConsumer.setMessageListener(m_msgListener);
            m_Connection.start();I feel like I've done my good dead for the day :)
-Jerome BG