Skip to Main Content

MDB with CMT rollback and redelivery despite getRollbackOnly being false.

tizzoMay 30 2012 — edited May 31 2012
I've got a message-driven bean using container managed transactions. Message processing involves some operations that can throw EJB exceptions, but those exceptions are caught and handled internally.

For some of these exceptions, my transaction commits and all is well, which is what I expect. But for other exceptions, the transaction is rolled back and the message redelivered, which is causing me problems. Worse, my efforts to code around these problems involve using getRollbackOnly() before returning from onMessage() to try to detect an impending rollback and prepare accordingly - however getRollbackOnly() invariably returns false - even if the transaction ends up rolling back an the message gets redelivered.

While it's not completely clear, the Javadoc for [url http://docs.oracle.com/javaee/6/api/javax/ejb/EJBContext.html#getRollbackOnly()]getRollbackOnly() strongly implies that it should return true whether rollbackOnly was requested by the application, or set automatically by an exception.

So my first question is - should it be possible for the container to roll back the transaction without telling me it's going to do so through getRollbackOnly(), or is this a bug? And if it's not a bug, is there some other way for me to know whether my transaction will commit or not?

I'm using Glassfish 3.1 with the built-in JMS provider. JSE6 on RHEL5.
@Resource
private MessageDrivenContext mdbContext;

public void onMessage(final Message inMessage) {
    try {
        // Sometimes throws, exception caught, transaction commits, no problem.
        doSomethingThatThrows();

        // Sometimes throws, exception caught and handled, but transaction rolled back and msg redelivered.
        doSomethingElseThatThrows();
    } catch (JMSException ex) {
        logExceptionMessage(ex);
    } finally {
        boolean rollback = mdbContext.getRollbackOnly();
        logRollbackValue(rollback);  // ALWAYS false.
    }
}
Edited by: 937599 on May 30, 2012 11:11 AM
Comments
Locked Post
New comments cannot be posted to this locked post.
Post Details
Locked due to inactivity on Jun 28 2012
Added on May 30 2012
2 comments
1,861 views