[ https://issues.apache.org/jira/browse/CAMEL-11777?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Claus Ibsen updated CAMEL-11777: -------------------------------- Fix Version/s: 2.20.0 > Transactional hazelcast:seda component uses not transaction aware queue > ----------------------------------------------------------------------- > > Key: CAMEL-11777 > URL: https://issues.apache.org/jira/browse/CAMEL-11777 > Project: Camel > Issue Type: Improvement > Components: camel-hazelcast > Affects Versions: 2.19.2 > Reporter: Kirill Merkushev > Fix For: 2.20.0 > > > As mentioned in docs > http://docs.hazelcast.org/docs/3.8.4/manual/html-single/index.html#creating-a-transaction-interface > Data structures should be transaction-aware and initialized inside of > transaction context. Now hazelcast seda component uses queue initialized > externally for the transaction. So when we rollback transaction - things stay > the same like we already committed changes (this simply noop). > Patch should look like (but exactly this version fails tests, so I should > investigate more) > {code:none} > index 7e3b24cc8f..cebd69ce16 100644 > --- > a/components/camel-hazelcast/src/main/java/org/apache/camel/component/hazelcast/seda/HazelcastSedaConsumer.java > +++ > b/components/camel-hazelcast/src/main/java/org/apache/camel/component/hazelcast/seda/HazelcastSedaConsumer.java > @@ -16,10 +16,10 @@ > */ > package org.apache.camel.component.hazelcast.seda; > > -import java.util.concurrent.BlockingQueue; > import java.util.concurrent.ExecutorService; > import java.util.concurrent.TimeUnit; > > +import com.hazelcast.core.BaseQueue; > import com.hazelcast.transaction.TransactionContext; > > import org.apache.camel.AsyncCallback; > @@ -71,7 +71,7 @@ public class HazelcastSedaConsumer extends DefaultConsumer > implements Runnable { > } > > public void run() { > - final BlockingQueue<?> queue = endpoint.getQueue(); > + BaseQueue<?> queue = > endpoint.getHazelcastInstance().getQueue(endpoint.getConfiguration().getQueueName()); > > while (queue != null && isRunAllowed()) { > final Exchange exchange = this.getEndpoint().createExchange(); > @@ -85,6 +85,7 @@ public class HazelcastSedaConsumer extends DefaultConsumer > implements Runnable { > if (transactionCtx != null) { > log.trace("Begin transaction: {}", > transactionCtx.getTxnId()); > transactionCtx.beginTransaction(); > + queue = > transactionCtx.getQueue(endpoint.getConfiguration().getQueueName()); > } > } > > {code} -- This message was sent by Atlassian JIRA (v6.4.14#64029)