You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@activemq.apache.org by tabish121 <gi...@git.apache.org> on 2016/06/20 15:38:31 UTC

[GitHub] activemq-artemis pull request #593: ARTEMIS-585 support send on dynamic send...

GitHub user tabish121 opened a pull request:

    https://github.com/apache/activemq-artemis/pull/593

    ARTEMIS-585 support send on dynamic sender link

    The sender abstraction must be able to update its sender address in the
    case of dynamic senders whose target address is not set until the code
    initializes the link and creates a destination for it.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/tabish121/activemq-artemis ARTEMIS-585

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/activemq-artemis/pull/593.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #593
    
----
commit bab2438d64d375311ec20a1380b783d28b5c31f4
Author: Timothy Bish <ta...@gmail.com>
Date:   2016-06-20T15:34:49Z

    ARTEMIS-585 support send on dynamic sender link
    
    The sender abstraction must be able to update its sender address in the
    case of dynamic senders whose target address is not set until the code
    initializes the link and creates a destination for it.

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] activemq-artemis pull request #593: ARTEMIS-585 support send on dynamic send...

Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:

    https://github.com/apache/activemq-artemis/pull/593


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] activemq-artemis issue #593: ARTEMIS-585 support send on dynamic sender link

Posted by tabish121 <gi...@git.apache.org>.
Github user tabish121 commented on the issue:

    https://github.com/apache/activemq-artemis/pull/593
  
    I copied that bit from another test and didn't see how long the delay was, I've updated the PR to use 500 ms instead :)


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] activemq-artemis issue #593: ARTEMIS-585 support send on dynamic sender link

Posted by gaohoward <gi...@git.apache.org>.
Github user gaohoward commented on the issue:

    https://github.com/apache/activemq-artemis/pull/593
  
    @tabish121  Actually not hang, the receive timeout is 500 seconds, maybe we can reduce it to 5?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] activemq-artemis issue #593: ARTEMIS-585 support send on dynamic sender link

Posted by gaohoward <gi...@git.apache.org>.
Github user gaohoward commented on the issue:

    https://github.com/apache/activemq-artemis/pull/593
  
    hmm, when I run the test in failure condition (without fix) it shows the correct error message but the test seems hang there.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---