[
https://issues.apache.org/activemq/browse/CAMEL-757?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Vadim Chekan updated CAMEL-757:
-------------------------------
Attachment: patch.txt
Claus, thanks for your review and guidance.
Attached new diff.
Added logging in doStart()
I do understand the issue with exceptions now. processPacket() which sends a
message is invoked asynchronously from smack library. And it is defined as not
throwing any exceptions because there is nobody up in the stack to handle it.
It is tempting to add exception handling to camel process() call but it would
be a bad idea. Sometimes endpoint needs information about success/failure for
example jdbc endpoint would rollback transaction if message have not been sent.
So I left try/catch but I replaced e.printStackTrace with LOG.error.
> Add room support in consumer mode
> ---------------------------------
>
> Key: CAMEL-757
> URL: https://issues.apache.org/activemq/browse/CAMEL-757
> Project: Apache Camel
> Issue Type: Improvement
> Components: camel-xmpp
> Affects Versions: 1.4.0
> Reporter: Vadim Chekan
> Priority: Minor
> Fix For: 1.5.0
>
> Attachments: patch.txt
>
>
> Currently XmppEndpoint support room only for producer but not for consumer.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
