Author: atm
Date: Fri Jan 25 00:44:37 2013
New Revision: 1438280

URL: http://svn.apache.org/viewvc?rev=1438280&view=rev
Log:
HDFS-4440. Avoid annoying log message when dfs.domain.socket.path is not set. 
Contributed by Colin Patrick McCabe.

Modified:
    
hadoop/common/branches/HDFS-347/hadoop-hdfs-project/hadoop-hdfs/CHANGES.HDFS-347.txt
    
hadoop/common/branches/HDFS-347/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/DomainSocketFactory.java

Modified: 
hadoop/common/branches/HDFS-347/hadoop-hdfs-project/hadoop-hdfs/CHANGES.HDFS-347.txt
URL: 
http://svn.apache.org/viewvc/hadoop/common/branches/HDFS-347/hadoop-hdfs-project/hadoop-hdfs/CHANGES.HDFS-347.txt?rev=1438280&r1=1438279&r2=1438280&view=diff
==============================================================================
--- 
hadoop/common/branches/HDFS-347/hadoop-hdfs-project/hadoop-hdfs/CHANGES.HDFS-347.txt
 (original)
+++ 
hadoop/common/branches/HDFS-347/hadoop-hdfs-project/hadoop-hdfs/CHANGES.HDFS-347.txt
 Fri Jan 25 00:44:37 2013
@@ -37,3 +37,5 @@ HDFS-4417. Fix case where local reads ge
 HDFS-4433. Make TestPeerCache not flaky (Colin Patrick McCabe via todd)
 
 HDFS-4438. TestDomainSocket fails when system umask is set to 0002. (Colin 
Patrick McCabe via atm)
+
+HDFS-4440. Avoid annoying log message when dfs.domain.socket.path is not set. 
(Colin Patrick McCabe via atm)

Modified: 
hadoop/common/branches/HDFS-347/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/DomainSocketFactory.java
URL: 
http://svn.apache.org/viewvc/hadoop/common/branches/HDFS-347/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/DomainSocketFactory.java?rev=1438280&r1=1438279&r2=1438280&view=diff
==============================================================================
--- 
hadoop/common/branches/HDFS-347/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/DomainSocketFactory.java
 (original)
+++ 
hadoop/common/branches/HDFS-347/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/DomainSocketFactory.java
 Fri Jan 25 00:44:37 2013
@@ -84,7 +84,7 @@ class DomainSocketFactory {
   DomainSocket create(InetSocketAddress addr, DFSInputStream stream) {
     // If there is no domain socket path configured, we can't use domain
     // sockets.
-    if (conf.domainSocketPath == null) return null;
+    if (conf.domainSocketPath.isEmpty()) return null;
     // UNIX domain sockets can only be used to talk to local peers
     if (!DFSClient.isLocalAddress(addr)) return null;
     // If the DomainSocket code is not loaded, we can't create


Reply via email to