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

svn commit: r1593647 - /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/Utilities.java

Author: hashutosh
Date: Fri May  9 23:23:07 2014
New Revision: 1593647

URL: http://svn.apache.org/r1593647
Log:
HIVE-7031 : Utiltites.createEmptyFile uses File.Separator instead of Path.Separator to create an empty file in HDFS (Hari Sankar via Ashutosh Chauhan)

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

Modified: hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/Utilities.java
URL: http://svn.apache.org/viewvc/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/Utilities.java?rev=1593647&r1=1593646&r2=1593647&view=diff
==============================================================================
--- hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/Utilities.java (original)
+++ hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/Utilities.java Fri May  9 23:23:07 2014
@@ -3046,7 +3046,7 @@ public final class Utilities {
           throws IOException, InstantiationException, IllegalAccessException {
 
     // create a dummy empty file in a new directory
-    String newDir = hiveScratchDir + File.separator + sequenceNumber;
+    String newDir = hiveScratchDir + Path.SEPARATOR + sequenceNumber;
     Path newPath = new Path(newDir);
     FileSystem fs = newPath.getFileSystem(job);
     fs.mkdirs(newPath);
@@ -3054,7 +3054,7 @@ public final class Utilities {
     //in the file status. This makes sure that all paths which goes into PathToPartitionInfo are always listed status
     //file path.
     newPath = fs.makeQualified(newPath);
-    String newFile = newDir + File.separator + "emptyFile";
+    String newFile = newDir + Path.SEPARATOR + "emptyFile";
     Path newFilePath = new Path(newFile);
 
     RecordWriter recWriter = outFileFormat.newInstance().getHiveRecordWriter(job, newFilePath,