[ https://issues.apache.org/jira/browse/CAMEL-9606?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15585549#comment-15585549 ]
ASF GitHub Bot commented on CAMEL-9606: --------------------------------------- GitHub user igarashitm opened a pull request: https://github.com/apache/camel/pull/1214 CAMEL-9606 Share JMS session among SJMS endpoints to ensure transacti… …on atomicity A testcase, which terminate a camel process during processing transacted sjms consumer/producer. Sometimes test-in + test-out > initial without the fix. https://github.com/igarashitm/issues/blob/master/camel/misc/src/test/java/CamelSjmsTxTest.java You can merge this pull request into a Git repository by running: $ git pull https://github.com/igarashitm/camel CAMEL-9606 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/camel/pull/1214.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #1214 ---- commit 825a35fbf67bc1b298bf8f9d58a5e2603d65063d Author: Tomohisa Igarashi <tm.igara...@gmail.com> Date: 2016-10-18T13:05:54Z CAMEL-9606 Share JMS session among SJMS endpoints to ensure transaction atomicity ---- > SJMS Consumer-Producer in transaciton > ------------------------------------- > > Key: CAMEL-9606 > URL: https://issues.apache.org/jira/browse/CAMEL-9606 > Project: Camel > Issue Type: Bug > Components: camel-sjms > Affects Versions: 2.15.4, 2.16.2 > Reporter: Zdeněk Obst > > I'm not 100% sure this is a bug but it feels that way from conversation I had > via mailing lists. > I'm trying to ensure transactional processing between SJMS consumer and > producer (e.g. using same JMS session). > In other words this simple case: > 1. prepare higher amount of JMS messages in broker (e.g. ActiveMQ with 1000 > messages) > 2. use Camel route from input queue to output queue using trasacted=true > 3. start context (starts consuming messages) and in any time kill java > process > When I kill process, I would expect that sum of messages in input and output > queue will be 1000 - so the transaction works. But what happens is that I > always end up with 1001+ messages. Maybe it is misconfiguration of routes or > misunderstanding how SJMS can work. > I feel this is critical because JMS is generally used because it its > transactional capabilities. > Here is the sample code I used for reproduction (using ActiveMQ): > {code:java} > public class SjmsTransaction { > public static void main(String[] args) throws Exception { > RouteBuilder rb = new RouteBuilder() { > @Override > public void configure() throws Exception { > onException(Exception.class) > .process(systemOut("Exception!!")); > from("sjms:queue:test-in?transacted=true&consumerCount=5") > .process(systemOut("Processing")) > .to("sjms:queue:test-out?transacted=true") > .process(systemOut("Processed")); > } > }; > CamelContext context = new DefaultCamelContext(); > addJmsComponent(context); > context.addRoutes(rb); > System.out.println("=====> Starting context"); > context.start(); > // Now the context will run and consume messages, when I kill > application by force in any time > // I expect this to be true: <#messagesInInputAtBeginning> == > <#messagesInInputNow> + <#messagesInOutputNow> > // What happens is that there is always < (e.g. I submitted 1000 > messages, out has 500, in has 501) > } > private static void addJmsComponent(CamelContext context) { > ConnectionFactory factory = new > ActiveMQConnectionFactory("tcp://localhost:61616"); > ConnectionFactoryResource connResource = new > ConnectionFactoryResource(5, factory); > SjmsComponent comp = new SjmsComponent(); > comp.setConnectionResource(connResource); > context.addComponent("sjms", comp); > } > private static Processor systemOut(final String message) { > return new Processor() { > @Override > public void process(Exchange exchange) throws Exception { > System.out.println(exchange.getExchangeId() + ": " + message); > } > }; > } > } > {code} > Note that I tried to use it with various combinations of acknowledgeMode and > In/InOut exchange pattern - but without luck. > I'm not that much oriented in Camel source code but I found that JMS session > is held within the exchange so probably when producer finds in an exchange > existing JMS session and is configured to be transacted, then maybe it can > participate this session? Or maybe there are other hooks (like > Synchronization objects) in some registry that take care of this issue? > Here is the link to the previous mailing list conversation: > http://camel.465427.n5.nabble.com/SJMS-transaction-td5777522.html -- This message was sent by Atlassian JIRA (v6.3.4#6332)