Discovery: all subclasses should only return a list of PhysicalAddresses instead of doing
the sending themselves
----------------------------------------------------------------------------------------------------------------
Key: JGRP-1353
URL:
https://issues.jboss.org/browse/JGRP-1353
Project: JGroups
Issue Type: Enhancement
Reporter: Bela Ban
Assignee: Bela Ban
Priority: Minor
Fix For: 3.0
Currently, Discovery.sendGetMembersRequest() is implemented separately in all subclasses
of Discovery. However, the code in it is almost the same; we grab a list of
PhysicalAddresses and send a discovery request to every element.
This functionality should be moved into Discovery, and every subclass should only (1)
return the list of PhysicalAddresses and (2) override a method getAddresses().
Method Discovery.sendDiscoveryRequest() would then call getAddresses() and send a
discovery request to every element. We also need to add a method
sendDiscoveryRequestsInParallel() to Discovery, which is overridden in every subclass, and
determines whether to use a separate timer thread for each discovery request, or not.
Note that getAddresses() may set metadata, too, e.g. in TCPGOSSIP, where it fetches
metadata (logical name, physical address, UUID) from the GossipRouter and sets it in its
local cache
--
This message is automatically generated by JIRA.
For more information on JIRA, see:
http://www.atlassian.com/software/jira