AsyncBaseLifeCycle. pollDeliveryChannel() calls processExchangeInTx regardless of exchange being in Transaction or not. After looking at the code it seems it does take care of both cases however, it would be easier to separate it and handle it separately than adding too many "if" condition for transaction check when the exchange is non-transacted.

This also confuses the users when they see some failures and exception stack trace shows reference to transaction.

It would be good to have it separated as it won't harm much in terms of performance and will be much better in case of failure stack trace analysis.