You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by GitBox <gi...@apache.org> on 2022/01/29 16:26:35 UTC

[GitHub] [camel-quarkus] zhfeng edited a comment on issue #3471: Transacted routing in combination with Split EIP fails

zhfeng edited a comment on issue #3471:
URL: https://github.com/apache/camel-quarkus/issues/3471#issuecomment-1024941928


   @bvahdat I agree with @jamesnetherton and  it could be related to https://github.com/apache/camel/commit/961ad0e56e9331e71c386415ec67676e586ea629 which introduce  ```void scheduleQueue(Runnable runnable)``` used for routing Exchange using transactions.
   
   Also It looks like the camel-spring and camel-jta has a slight different with ```TransactionErrorHandler::processByErrorHandler(final Exchange exchange)```
   
   https://github.com/apache/camel/blob/main/components/camel-spring/src/main/java/org/apache/camel/spring/spi/TransactionErrorHandler.java#L243-L250
   ```java
   protected void processByErrorHandler(final Exchange exchange) {
           awaitManager.process(new AsyncProcessorSupport() {
               @Override
               public boolean process(Exchange exchange, AsyncCallback callback) {
                   return TransactionErrorHandler.super.process(exchange, callback);
               }
           }, exchange);
       }
   ```
   
   https://github.com/apache/camel/blob/main/components/camel-jta/src/main/java/org/apache/camel/jta/TransactionErrorHandler.java#L231-L237
   ```java
   protected void processByErrorHandler(final Exchange exchange) {
           try {
               output.process(exchange);
           } catch (Throwable e) {
               throw new RuntimeCamelException(e);
           }
       }
   ```
   
   That maybe be the reason that your reproducer is failing with camel-spring.
   
   I'm trying to reproduce this issue with ```camel-jta``` and I belive this should be fixed in the upstream Camel but not CQ.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org