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 2013/12/20 17:50:21 UTC
svn commit: r1552733 -
/hive/trunk/itests/util/src/main/java/org/apache/hadoop/hive/ql/QTestUtil.java
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() {