honeymoose
/
OpenSearch
mirror of https://github.com/honeymoose/OpenSearch.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Remove unnecessary field from UnicastZenPing 

This field was only ever used in the constructor, where it was set and
then passed around. As such, there's no need for it to be a field and we
can remove it.
This commit is contained in:
Jason Tedor 2017-05-03 08:25:14 -04:00
parent a45e2efa00
commit 16af0a9ce2
1 changed files with 1 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
core/src/main/java/org/elasticsearch/discovery/zen/UnicastZenPing.java
View File

@ -111,8 +111,6 @@ public class UnicastZenPing extends AbstractComponent implements ZenPing {
private final TransportService transportService;
private final ClusterName clusterName;
private final int concurrentConnects;
private final List<String> configuredHosts;
private final int limitPortCounts;
@ -145,7 +143,7 @@ public class UnicastZenPing extends AbstractComponent implements ZenPing {
this.clusterName = ClusterName.CLUSTER_NAME_SETTING.get(settings);
this.hostsProvider = unicastHostsProvider;
this.concurrentConnects = DISCOVERY_ZEN_PING_UNICAST_CONCURRENT_CONNECTS_SETTING.get(settings);
final int concurrentConnects = DISCOVERY_ZEN_PING_UNICAST_CONCURRENT_CONNECTS_SETTING.get(settings);
if (DISCOVERY_ZEN_PING_UNICAST_HOSTS_SETTING.exists(settings)) {
configuredHosts = DISCOVERY_ZEN_PING_UNICAST_HOSTS_SETTING.get(settings);
// we only limit to 1 addresses, makes no sense to ping 100 ports