[
https://issues.apache.org/jira/browse/QPID-7874?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16108902#comment-16108902
]
ASF GitHub Bot commented on QPID-7874:
--------------------------------------
GitHub user chrisrichardson77 opened a pull request:
https://github.com/apache/qpid-cpp/pull/5
QPID-7874 Remove unnecessary link check when adding routes with qpid-…
See https://issues.apache.org/jira/browse/QPID-7874
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/fourceu/qpid-cpp qpid-route-no-link-checks
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/qpid-cpp/pull/5.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 #5
----
commit 7aae3a0f2f00fe916b76f67113b9e9a8699784c1
Author: Chris Richardson <[email protected]>
Date: 2017-08-01T13:37:30Z
QPID-7874 Remove unnecessary link check when adding routes with qpid-route
----
> qpid-route performs an unnecessary link check when adding routes
> ----------------------------------------------------------------
>
> Key: QPID-7874
> URL: https://issues.apache.org/jira/browse/QPID-7874
> Project: Qpid
> Issue Type: Improvement
> Components: C++ Broker
> Affects Versions: qpid-cpp-1.36.0
> Reporter: Chris Richardson
>
> Having connected to the destination broker (for "normal" routes) or source
> broker (for "src-local" routes), the qpid-route utility checks that the
> inter-broker link is active before configuring the new bridge. This check is
> not necessary since the bridge can still be created even if the network link
> is down. When the remote broker becomes accessible, the inter-broker link and
> the bridge will come up as expected.
> PR:
> An alternative to simply removing the check might be to make it a
> configuration option.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
