You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hive.apache.org by br...@apache.org on 2014/09/10 18:01:01 UTC

svn commit: r1624045 - /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java

Author: brock
Date: Wed Sep 10 16:01:00 2014
New Revision: 1624045

URL: http://svn.apache.org/r1624045
Log:
HIVE-8034 - Don't add colon when no port is specified (Brock reviewed by Ashutosh)

Modified:
    hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java

Modified: hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java
URL: http://svn.apache.org/viewvc/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java?rev=1624045&r1=1624044&r2=1624045&view=diff
==============================================================================
--- hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java (original)
+++ hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java Wed Sep 10 16:01:00 2014
@@ -1351,9 +1351,9 @@ public class DDLTask extends Task<DDLWor
         if(harPartitionDir.getUserInfo() != null) {
           authority.append(harPartitionDir.getUserInfo()).append("@");
         }
-        authority.append(harPartitionDir.getHost()).append(":");
+        authority.append(harPartitionDir.getHost());
         if(harPartitionDir.getPort() != -1) {
-          authority.append(harPartitionDir.getPort());
+          authority.append(":").append(harPartitionDir.getPort());
         }
         Path harPath = new Path(harPartitionDir.getScheme(),
             authority.toString(),