[
https://issues.apache.org/jira/browse/QPID-3767?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13189129#comment-13189129
]
[email protected] commented on QPID-3767:
-----------------------------------------------------
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/3546/
-----------------------------------------------------------
Review request for qpid, Alan Conway, Gordon Sim, michael goulish, and Ted Ross.
Summary
-------
This patch modifies the way the broker's Link and Bridge objects are identified
and managed. Specifically:
1) both Bridge and Links are now identified by explict names assigned by
management, rather than destination host/port info.
- names beginning with the prefix "qpid." are reserved for qpidd internal
use.
- for backward compatibility, if no name is assigned on creation, the broker
will generate a name based on UUID
2) the corresponding QMF objects have been updated accordingly, with the
additions of:
- the QMF Link object has been updated to provide a reference to the
corresponding Connection
- the QMF Link object has been modified to allow the host/port/connectionRef
to change on failover
- the QMF Bridge object has been modified to allow the Channel identifier to
change (allowing Bridges to be reassigned to different links in the future)
3) Links/Bridges may now be created/deleted via the QMF Broker's generic
"create" and "delete" methods
4) Some consolidation of the Link/Bridge creation APIs, specifically:
- Link/Bridges are created via calls to the LinkRegistry's "declare()"
methods
- Link/Bridges are removed by calling their corresponding "destroy()" methods
More importantly, the above changes make it possible to create multiple Links
between the same two brokers. This can be done by creating Links to the same
destinations with different names. This is a change from the existing
behavior, which uses the destination host/port as the unique Link identifier.
This addresses bug qpid-3767.
https://issues.apache.org/jira/browse/qpid-3767
Diffs
-----
/trunk/qpid/cpp/src/qpid/broker/Bridge.h 1233125
/trunk/qpid/cpp/src/qpid/broker/Bridge.cpp 1233125
/trunk/qpid/cpp/src/qpid/broker/Broker.cpp 1233125
/trunk/qpid/cpp/src/qpid/broker/Connection.cpp 1233125
/trunk/qpid/cpp/src/qpid/broker/Link.h 1233125
/trunk/qpid/cpp/src/qpid/broker/Link.cpp 1233125
/trunk/qpid/cpp/src/qpid/broker/LinkRegistry.h 1233125
/trunk/qpid/cpp/src/qpid/broker/LinkRegistry.cpp 1233125
/trunk/qpid/cpp/src/tests/federation.py 1233125
/trunk/qpid/specs/management-schema.xml 1233125
Diff: https://reviews.apache.org/r/3546/diff
Testing
-------
This patch fails to pass some of the cluster tests - I'm investigating this
now. All non-cluster federation tests where passing (prior to my latest
rebase).
Work remains, but I wanted to get this patch out for discussion before going
much farther.
Thanks,
Kenneth
> Federation link index becomes invalid on failover against a cluster.
> --------------------------------------------------------------------
>
> Key: QPID-3767
> URL: https://issues.apache.org/jira/browse/QPID-3767
> Project: Qpid
> Issue Type: Bug
> Components: C++ Broker
> Affects Versions: 0.14
> Reporter: Ken Giusti
> Assignee: Ken Giusti
> Fix For: 0.15
>
>
> The Link management object that represents a connection between two federated
> brokers is indexed (identified) by the remote broker's host and port. If the
> remote broker is part of a cluster, and a failover event occurs, the
> host:port used by the Link object's index may no longer exist. This prevents
> the route from being deleted.
> For example, create a cluster of two brokers using addresses 127.0.0.1:2222
> and 127.0.0.1:3333. Start a third broker, say 127.0.0.1:8888. Create a
> queue route from 127.0.0.1:2222 to an exchange on 127.0.0.1:8888. Kill the
> broker 127.0.0.1:2222. This results in a Link object that is connected to
> 127.0.0.1:3333, but reports 127.0.0.1:2222 as it's host.
> [kgiusti@localhost src]$ qpid-config -a 127.0.0.1:2222 add queue src
> [kgiusti@localhost src]$ qpid-config -a 127.0.0.1:8888 add exchange fanout
> destx
> [kgiusti@localhost src]$ qpid-config -a 127.0.0.1:8888 add queue dest
> [kgiusti@localhost src]$ qpid-config -a 127.0.0.1:8888 bind destx dest
> [kgiusti@localhost src]$ qpid-route queue add 127.0.0.1:8888 127.0.0.1:2222
> destx src
> [kgiusti@localhost src]$ ../examples/messaging/spout -b 127.0.0.1:2222
> --content "ZZZ" src
> [kgiusti@localhost src]$ ../examples/messaging/drain -b 127.0.0.1:8888 -t 2
> dest
> Message(properties={spout-id:8c308c74-6b25-4408-8694-93ef8352a308:0,
> x-amqp-0-10.routing-key:src}, content='ZZZ')
> <Kill Broker 127.0.0.1:2222, link fails over to 127.0.0.1:3333>
> From qpid-tool:
> qpid: show 133
> Object of type:
> org.apache.qpid.broker:link:_data(bc33c1b3-25cd-e0ce-04d7-ad684ed36d91)
> Attribute 133
> =================================================
> vhostRef 150
> host 127.0.0.1
> port 2222
> transport tcp
> durable False
> state Operational
> lastError Failed over to tcp:10.16.185.15:3333
> Once this occurs, I am unable to delete the link:
> [kgiusti@localhost src]$ qpid-route queue del 127.0.0.1:8888 127.0.0.1:2222
> destx src
> [kgiusti@localhost src]$ qpid-tool 127.0.0.1:8888
> qpid: list
> Summary of Objects by Type:
> Package Class Active Deleted
> =======================================================
> org.apache.qpid.broker binding 14 0
> org.apache.qpid.broker system 1 0
> org.apache.qpid.broker broker 1 0
> org.apache.qpid.broker bridge 1 0
> org.apache.qpid.broker link 1 0
> org.apache.qpid.broker subscription 5 0
> org.apache.qpid.broker connection 2 0
> org.apache.qpid.broker session 2 0
> org.apache.qpid.broker queue 6 0
> org.apache.qpid.broker exchange 9 0
> org.apache.qpid.broker vhost 1 0
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
---------------------------------------------------------------------
Apache Qpid - AMQP Messaging Implementation
Project: http://qpid.apache.org
Use/Interact: mailto:[email protected]