[jira] [Commented] (APLO-6) Alternative topic separators should be supported in Apollo, as in ActiveMQ

["Hiram Chirino (JIRA)" <ji...@apache.org>]

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

Hiram Chirino commented on APLO-6:
----------------------------------

Hey Sam,

Where do you think would be the best place to configure that option? A a broker level?  Per connector? Per client?

> Alternative topic separators should be supported in Apollo, as in ActiveMQ
> --------------------------------------------------------------------------
>
>                 Key: APLO-6
>                 URL: https://issues.apache.org/jira/browse/APLO-6
>             Project: ActiveMQ Apollo
>          Issue Type: Bug
>          Components: apollo-broker
>    Affects Versions: 1.0-beta2
>         Environment: All environments.
>            Reporter: Sam Starling
>            Assignee: Hiram Chirino
>              Labels: apollo, topic
>
> When using a character other than a period as a topic separator (.), Apollo throws the following exception:
> {code} 
> org.apache.activemq.apollo.util.path.PathParser$PathException: Invalid destination path part: 'event/abc', it does not match regex: [a-zA-Z0-9\_\-\%\~]+
> {code}
> In the example, we're using {{/topic/event/abc}} as a topic. In {{[PathParser.java|http://svn.apache.org/viewvc/activemq/activemq-apollo/trunk/apollo-util/src/main/scala/org/apache/activemq/apollo/util/path/PathParser.java?view=markup]}}, line 47 determines that {{path_seperator = new AsciiBuffer(".")}}. Thus, Apollo incorrectly tokenizes the topic and so {{event/abc}} fails the regular expression on line 48 of the same file.
> In short, it should be possible to specify what the topic separator is, or (even better) detect when a different character is being used as a delimiter.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira