camel-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Luca Burgazzoli (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (CAMEL-10757) camel-jms - Use dash in destination type enum instead of colon
Date Wed, 08 Feb 2017 17:20:41 GMT

    [ https://issues.apache.org/jira/browse/CAMEL-10757?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15858264#comment-15858264
] 

Luca Burgazzoli commented on CAMEL-10757:
-----------------------------------------

For what I can see, the points were the colon is explicit used are:

https://github.com/apache/activemq/blob/master/activemq-camel/src/main/java/org/apache/activemq/camel/component/broker/BrokerComponent.java#L78-L83
https://github.com/apache/activemq/blob/master/activemq-camel/src/main/java/org/apache/activemq/camel/component/ActiveMQComponent.java#L282-L287

Unfortunately activemq has camel 2.16 as dependency so I can't easily run its test suite (as
it requires camel-spring3).
One more reason to move it under camel ...


> camel-jms - Use dash in destination type enum instead of colon 
> ---------------------------------------------------------------
>
>                 Key: CAMEL-10757
>                 URL: https://issues.apache.org/jira/browse/CAMEL-10757
>             Project: Camel
>          Issue Type: Improvement
>          Components: camel-jms
>            Reporter: Claus Ibsen
>            Assignee: Luca Burgazzoli
>            Priority: Minor
>             Fix For: 2.19.0
>
>
> Instead of using temp:queue or temp:topic we should use temp-queue or temp-topic which
wont cause problem with the colon separator in the uri syntax.
> This also aids tooling and camel-catalog to not have complex parsing due the colon problem.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Mime
View raw message