Github user PepperJo commented on a diff in the pull request:

    https://github.com/apache/incubator-crail/pull/25#discussion_r196677357
  
    --- Diff: 
storage-narpc/src/main/java/org/apache/crail/storage/tcp/TcpStorageServer.java 
---
    @@ -176,13 +176,19 @@ public static InetSocketAddress getDataNodeAddress() 
throws IOException {
                                InetAddress _addr = address.getAddress();
                                addr = _addr;
                        }
    -           }               
    +           }
    +
    +           if (addr == null){
    +                   throw new IOException("Cannot find valid network 
interface with name " + TcpStorageConstants.STORAGE_TCP_INTERFACE);
    --- End diff --
    
    The message is a bit vague. Why not: "Network interface with name XXX has 
no valid IP address"


---

Reply via email to