review fixes

Author: NSAmelchev

modules/core/src/main/java/org/apache/ignite/internal/managers/discovery/DiscoveryMessageFactory.java

@@ -52,8 +52,8 @@
 public class DiscoveryMessageFactory implements MessageFactoryProvider {
     /** {@inheritDoc} */
     @Override public void registerAll(MessageFactory factory) {
-        factory.register((short)-101, InetAddressMessage::new, new InetAddressMessageSerializer());
-        factory.register((short)-100, InetSocketAddressMessage::new, new InetSocketAddressMessageSerializer());
+        factory.register((short)-101, InetSocketAddressMessage::new, new InetSocketAddressMessageSerializer());
+        factory.register((short)-100, InetAddressMessage::new, new InetAddressMessageSerializer());
 
         factory.register((short)0, TcpDiscoveryCheckFailedMessage::new, new TcpDiscoveryCheckFailedMessageSerializer());
         factory.register((short)1, TcpDiscoveryPingRequest::new, new TcpDiscoveryPingRequestSerializer());

modules/core/src/main/java/org/apache/ignite/spi/discovery/tcp/TcpDiscoverySpi.java

@@ -2019,7 +2019,7 @@ protected IgniteSpiException authenticationFailedError(TcpDiscoveryAuthFailedMes
         assert msg != null;
 
         return new IgniteSpiException(new IgniteAuthenticationException("Authentication failed [nodeId=" +
-            msg.creatorNodeId() + ", addr=" + msg.address().getHostAddress() + ']'));
+            msg.creatorNodeId() + ", addr=" + msg.creatorAddress().getHostAddress() + ']'));
     }
 
     /**

modules/core/src/main/java/org/apache/ignite/spi/discovery/tcp/messages/InetSocketAddressMessage.java

@@ -22,7 +22,7 @@
 import org.apache.ignite.internal.managers.discovery.DiscoveryMessageFactory;
 import org.apache.ignite.internal.util.typedef.internal.S;
 
-/** Address utility container message. Is not a pure {@link TcpDiscoveryAbstractMessage}. */
+/** Socket address utility container message. Is not a pure {@link TcpDiscoveryAbstractMessage}. */
 public class InetSocketAddressMessage extends InetAddressMessage {
     /** */
     @Order(2)
@@ -47,12 +47,12 @@ public InetSocketAddressMessage(InetAddress addr, int port) {
         this.port = port;
     }
 
-    /** @return port. */
+    /** @return Port. */
     public int port() {
         return port;
     }
 
-    /** @param port port. */
+    /** @param port Port. */
     public void port(int port) {
         this.port = port;
     }

modules/core/src/main/java/org/apache/ignite/spi/discovery/tcp/messages/TcpDiscoveryAuthFailedMessage.java

@@ -25,15 +25,15 @@
 import org.apache.ignite.plugin.extensions.communication.Message;
 
 /**
- * Message telling joining node that its authentication failed on coordinator.
+ * Message telling joining node that its authentication failed.
  */
 public class TcpDiscoveryAuthFailedMessage extends TcpDiscoveryAbstractMessage implements Message {
     /** */
     private static final long serialVersionUID = 0L;
 
-    /** Coordinator address. */
-    @Order(value = 5, method = "addressMessage")
-    private InetAddressMessage addr;
+    /** Creator address. */
+    @Order(value = 5, method = "creatorAddressMessage")
+    private InetAddressMessage creatorAddrMsg;
 
     /** Node id for which authentication was failed. */
     @Order(6)
@@ -48,13 +48,13 @@ public TcpDiscoveryAuthFailedMessage() {
      * Constructor.
      *
      * @param creatorNodeId Creator node ID.
-     * @param addr Coordinator address.
+     * @param creatorAddr Creator address.
      * @param targetNodeId Node for which authentication was failed.
      */
-    public TcpDiscoveryAuthFailedMessage(UUID creatorNodeId, InetAddress addr, UUID targetNodeId) {
+    public TcpDiscoveryAuthFailedMessage(UUID creatorNodeId, InetAddress creatorAddr, UUID targetNodeId) {
         super(creatorNodeId);
 
-        this.addr = new InetAddressMessage(addr);
+        this.creatorAddrMsg = new InetAddressMessage(creatorAddr);
         this.targetNodeId = targetNodeId;
     }
 
@@ -68,19 +68,19 @@ public void targetNodeId(UUID targetNodeId) {
         this.targetNodeId = targetNodeId;
     }
 
-    /** @return Coordinator address. */
-    public InetAddressMessage addressMessage() {
-        return addr;
+    /** @return Creator address message. */
+    public InetAddressMessage creatorAddressMessage() {
+        return creatorAddrMsg;
     }
 
-    /** @param addr Coordinator address. */
-    public void addressMessage(InetAddressMessage addr) {
-        this.addr = addr;
+    /** @param addr Creator address message. */
+    public void creatorAddressMessage(InetAddressMessage addr) {
+        this.creatorAddrMsg = addr;
     }
 
-    /** @return Coordinator address. */
-    public InetAddress address() {
-        return addr.address();
+    /** @return Creator address. */
+    public InetAddress creatorAddress() {
+        return creatorAddrMsg.address();
     }
 
     /** {@inheritDoc} */