[GitHub] [ignite] Vladsz83 commented on a change in pull request #8424: IGNITE-13663 : Represent in the documenttion affection of several node addresses on failure detection v2.

[GitBox <gi...@apache.org>]

Vladsz83 commented on a change in pull request #8424:
URL: https://github.com/apache/ignite/pull/8424#discussion_r526346857



##########
File path: docs/_docs/clustering/network-configuration.adoc
##########
@@ -49,10 +49,18 @@ tab:C++[unsupported]
 The following table describes some most important properties of `TcpDiscoverySpi`.
 You can find the complete list of properties in the javadoc:org.apache.ignite.spi.discovery.tcp.TcpDiscoverySpi[] javadoc.
 
+[CAUTION]
+====
+You should assign `IgniteConfiguration.localHost` or `TcpDiscoverySpi.localAddress`.

Review comment:
       Fixed with the exception we can bind only to one net. interface currently. Or automatically to all.

##########
File path: docs/_docs/clustering/network-configuration.adoc
##########
@@ -49,10 +49,18 @@ tab:C++[unsupported]
 The following table describes some most important properties of `TcpDiscoverySpi`.
 You can find the complete list of properties in the javadoc:org.apache.ignite.spi.discovery.tcp.TcpDiscoverySpi[] javadoc.
 
+[CAUTION]
+====
+You should assign `IgniteConfiguration.localHost` or `TcpDiscoverySpi.localAddress`.
+By default, node obtains all available IP addresses. Several node addresses can prolong detection of node failure.
+You should not allow node bind to multiple ip addresses until they represent some real physical connections which can
+give you more reliability.
+====
+
 [cols="1,2,1",opts="header"]
 |===
 |Property | Description| Default Value
-| `localAddress`| Local host IP address used for discovery. | By default, the node uses the first non-loopback address it finds. If there is no non-loopback address available, then `java.net.InetAddress.getLocalHost()` is used.
+| `localAddress`| Local host IP address used for discovery. Prevails over `IgniteConfiguration.localHost`. | By default, all available addresses are assigned to node. If there is no non-loopback address available, then `java.net.InetAddress.getLocalHost()` is used.

Review comment:
       Fixed.

##########
File path: docs/_docs/clustering/network-configuration.adoc
##########
@@ -111,7 +119,7 @@ You can find the list of all properties in the javadoc:org.apache.ignite.spi.com
 [cols="1,2,1",opts="header"]
 |===
 |Property | Description| Default Value
-| `localAddress` | The local address for the communication SPI to bind to. |
+| `localAddress` | The local address for the communication SPI to bind to. Prevails over `IgniteConfiguration.localHost`. |

Review comment:
       Fixed.




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org