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 2012/11/08 17:33:47 UTC
svn commit: r1407159 - in
/hive/trunk/shims/src/common/java/org/apache/hadoop/fs:
ProxyFileSystem.java ProxyLocalFileSystem.java
Author: hashutosh
Date: Thu Nov 8 16:33:46 2012
New Revision: 1407159
URL: http://svn.apache.org/viewvc?rev=1407159&view=rev
Log:
HIVE-3661 : Remove the Windows specific = related swizzle path changes from Proxy FileSystems (Kanna Karanam via Ashutosh Chauhan)
Modified:
hive/trunk/shims/src/common/java/org/apache/hadoop/fs/ProxyFileSystem.java
hive/trunk/shims/src/common/java/org/apache/hadoop/fs/ProxyLocalFileSystem.java
Modified: hive/trunk/shims/src/common/java/org/apache/hadoop/fs/ProxyFileSystem.java
URL: http://svn.apache.org/viewvc/hive/trunk/shims/src/common/java/org/apache/hadoop/fs/ProxyFileSystem.java?rev=1407159&r1=1407158&r2=1407159&view=diff
==============================================================================
--- hive/trunk/shims/src/common/java/org/apache/hadoop/fs/ProxyFileSystem.java (original)
+++ hive/trunk/shims/src/common/java/org/apache/hadoop/fs/ProxyFileSystem.java Thu Nov 8 16:33:46 2012
@@ -47,22 +47,12 @@ public class ProxyFileSystem extends Fil
private Path swizzleParamPath(Path p) {
String pathUriString = p.toUri().toString();
- if (Shell.WINDOWS) {
- // Some of the file paths (Files with partition option) in HDFS '='
- // but Windows file path doesn't support '=' so replace it with special string.
- pathUriString = pathUriString.replaceAll("=", "------");
- }
URI newPathUri = URI.create(pathUriString);
return new Path (realScheme, realAuthority, newPathUri.getPath());
}
private Path swizzleReturnPath(Path p) {
String pathUriString = p.toUri().toString();
- if (Shell.WINDOWS) {
- // Revert back the special string '------' with '=' when we do the reverse conversion
- // from Windows path to HDFS
- pathUriString = pathUriString.replaceAll("------", "=");
- }
URI newPathUri = URI.create(pathUriString);
return new Path (myScheme, myAuthority, newPathUri.getPath());
}
Modified: hive/trunk/shims/src/common/java/org/apache/hadoop/fs/ProxyLocalFileSystem.java
URL: http://svn.apache.org/viewvc/hive/trunk/shims/src/common/java/org/apache/hadoop/fs/ProxyLocalFileSystem.java?rev=1407159&r1=1407158&r2=1407159&view=diff
==============================================================================
--- hive/trunk/shims/src/common/java/org/apache/hadoop/fs/ProxyLocalFileSystem.java (original)
+++ hive/trunk/shims/src/common/java/org/apache/hadoop/fs/ProxyLocalFileSystem.java Thu Nov 8 16:33:46 2012
@@ -53,11 +53,9 @@ public class ProxyLocalFileSystem extend
if (Shell.WINDOWS) {
// Replace the encoded backward slash with forward slash
// Remove the windows drive letter
- // replace the '=' with special string '------' to handle the unsupported char '=' in windows.
nameUriString = nameUriString.replaceAll("%5C", "/")
.replaceFirst("/[c-zC-Z]:", "/")
- .replaceFirst("^[c-zC-Z]:", "")
- .replaceAll("=", "------");
+ .replaceFirst("^[c-zC-Z]:", "");
name = URI.create(nameUriString);
}