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