You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@chukwa.apache.org by "Bill Graham (JIRA)" <ji...@apache.org> on 2010/08/09 18:53:20 UTC

[jira] Created: (CHUKWA-507) JMSAdaptor url validation too strict

JMSAdaptor url validation too strict
------------------------------------

                 Key: CHUKWA-507
                 URL: https://issues.apache.org/jira/browse/CHUKWA-507
             Project: Chukwa
          Issue Type: Improvement
          Components: data collection
            Reporter: Bill Graham
            Assignee: Bill Graham
            Priority: Minor


the JMSAdaptor requires that all JMS broker urls start with 'tcp://' or 'vm://', which is too strict. ActiveMQ has another transport that starts with 'failover://' for example.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (CHUKWA-507) JMSAdaptor url validation too strict

Posted by "Eric Yang (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/CHUKWA-507?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12898713#action_12898713 ] 

Eric Yang commented on CHUKWA-507:
----------------------------------

+1 looks good.

> JMSAdaptor url validation too strict
> ------------------------------------
>
>                 Key: CHUKWA-507
>                 URL: https://issues.apache.org/jira/browse/CHUKWA-507
>             Project: Chukwa
>          Issue Type: Improvement
>          Components: data collection
>            Reporter: Bill Graham
>            Assignee: Bill Graham
>            Priority: Minor
>         Attachments: CHUKWA-507-1.patch
>
>
> the JMSAdaptor requires that all JMS broker urls start with 'tcp://' or 'vm://', which is too strict. ActiveMQ has another transport that starts with 'failover://' for example.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (CHUKWA-507) JMSAdaptor url validation too strict

Posted by "Bill Graham (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/CHUKWA-507?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Bill Graham updated CHUKWA-507:
-------------------------------

    Attachment: CHUKWA-507-1.patch

Attaching patch that just requires '://' to be present.

> JMSAdaptor url validation too strict
> ------------------------------------
>
>                 Key: CHUKWA-507
>                 URL: https://issues.apache.org/jira/browse/CHUKWA-507
>             Project: Chukwa
>          Issue Type: Improvement
>          Components: data collection
>            Reporter: Bill Graham
>            Assignee: Bill Graham
>            Priority: Minor
>         Attachments: CHUKWA-507-1.patch
>
>
> the JMSAdaptor requires that all JMS broker urls start with 'tcp://' or 'vm://', which is too strict. ActiveMQ has another transport that starts with 'failover://' for example.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (CHUKWA-507) JMSAdaptor url validation too strict

Posted by "Bill Graham (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/CHUKWA-507?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Bill Graham updated CHUKWA-507:
-------------------------------

    Status: Patch Available  (was: Open)

> JMSAdaptor url validation too strict
> ------------------------------------
>
>                 Key: CHUKWA-507
>                 URL: https://issues.apache.org/jira/browse/CHUKWA-507
>             Project: Chukwa
>          Issue Type: Improvement
>          Components: data collection
>            Reporter: Bill Graham
>            Assignee: Bill Graham
>            Priority: Minor
>         Attachments: CHUKWA-507-1.patch
>
>
> the JMSAdaptor requires that all JMS broker urls start with 'tcp://' or 'vm://', which is too strict. ActiveMQ has another transport that starts with 'failover://' for example.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (CHUKWA-507) JMSAdaptor url validation too strict

Posted by "Eric Yang (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/CHUKWA-507?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Eric Yang updated CHUKWA-507:
-----------------------------

        Status: Resolved  (was: Patch Available)
    Resolution: Fixed

I just committed this, thanks Bill.

> JMSAdaptor url validation too strict
> ------------------------------------
>
>                 Key: CHUKWA-507
>                 URL: https://issues.apache.org/jira/browse/CHUKWA-507
>             Project: Chukwa
>          Issue Type: Improvement
>          Components: data collection
>            Reporter: Bill Graham
>            Assignee: Bill Graham
>            Priority: Minor
>         Attachments: CHUKWA-507-1.patch
>
>
> the JMSAdaptor requires that all JMS broker urls start with 'tcp://' or 'vm://', which is too strict. ActiveMQ has another transport that starts with 'failover://' for example.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.