Author: hashutosh
Date: Fri Dec 20 16:50:21 2013
New Revision: 1552733

URL: http://svn.apache.org/r1552733
Log:
HIVE-6064 : Wincompat: windows path substitutions overridden by 
MiniMrShim.getConfiguration() on hadoop-2 (Jason Dere via Ashutosh Chauhan)

Modified:
    
hive/trunk/itests/util/src/main/java/org/apache/hadoop/hive/ql/QTestUtil.java

Modified: 
hive/trunk/itests/util/src/main/java/org/apache/hadoop/hive/ql/QTestUtil.java
URL: 
http://svn.apache.org/viewvc/hive/trunk/itests/util/src/main/java/org/apache/hadoop/hive/ql/QTestUtil.java?rev=1552733&r1=1552732&r2=1552733&view=diff
==============================================================================
--- 
hive/trunk/itests/util/src/main/java/org/apache/hadoop/hive/ql/QTestUtil.java 
(original)
+++ 
hive/trunk/itests/util/src/main/java/org/apache/hadoop/hive/ql/QTestUtil.java 
Fri Dec 20 16:50:21 2013
@@ -231,10 +231,6 @@ public class QTestUtil {
 
   public void initConf() throws Exception {
 
-    if (Shell.WINDOWS) {
-      convertPathsFromWindowsToHdfs();
-    }
-
     String vectorizationEnabled = 
System.getProperty("test.vectorization.enabled");
     if(vectorizationEnabled != null && 
vectorizationEnabled.equalsIgnoreCase("true")) {
       conf.setBoolVar(ConfVars.HIVE_VECTORIZATION_ENABLED, true);
@@ -258,6 +254,12 @@ public class QTestUtil {
                   (new Path(dfsUriString,
                             "/build/ql/test/data/warehouse/")).toString());
     }
+    
+    // Windows paths should be converted after MiniMrShim.setupConfiguration()
+    // since setupConfiguration may overwrite configuration values.
+    if (Shell.WINDOWS) {
+      convertPathsFromWindowsToHdfs();
+    }
   }
 
   private void convertPathsFromWindowsToHdfs() {


Reply via email to