You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@drill.apache.org by pa...@apache.org on 2015/09/04 01:58:01 UTC

drill git commit: DRILL-3711: Fix unit tests on Windows

Repository: drill
Updated Branches:
  refs/heads/master 95623912e -> 9baec8a89


DRILL-3711: Fix unit tests on Windows


Project: http://git-wip-us.apache.org/repos/asf/drill/repo
Commit: http://git-wip-us.apache.org/repos/asf/drill/commit/9baec8a8
Tree: http://git-wip-us.apache.org/repos/asf/drill/tree/9baec8a8
Diff: http://git-wip-us.apache.org/repos/asf/drill/diff/9baec8a8

Branch: refs/heads/master
Commit: 9baec8a89d438fae2309091be82204c37447afee
Parents: 9562391
Author: Parth Chandra <pa...@apache.org>
Authored: Tue Aug 25 18:04:11 2015 -0700
Committer: Parth Chandra <pa...@apache.org>
Committed: Thu Sep 3 16:57:46 2015 -0700

----------------------------------------------------------------------
 .../org/apache/drill/exec/planner/sql/HivePartitionLocation.java | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/drill/blob/9baec8a8/contrib/storage-hive/core/src/main/java/org/apache/drill/exec/planner/sql/HivePartitionLocation.java
----------------------------------------------------------------------
diff --git a/contrib/storage-hive/core/src/main/java/org/apache/drill/exec/planner/sql/HivePartitionLocation.java b/contrib/storage-hive/core/src/main/java/org/apache/drill/exec/planner/sql/HivePartitionLocation.java
index e3066a4..8a99026 100644
--- a/contrib/storage-hive/core/src/main/java/org/apache/drill/exec/planner/sql/HivePartitionLocation.java
+++ b/contrib/storage-hive/core/src/main/java/org/apache/drill/exec/planner/sql/HivePartitionLocation.java
@@ -22,7 +22,8 @@ import org.apache.drill.exec.planner.PartitionLocation;
 public class HivePartitionLocation implements PartitionLocation {
   private final String partitionLocation;
   private final String[] partitionValue;
-  private final String fileSeparator = System.getProperty("file.separator");
+  // The path names passed in to this class are already sanitised and use the forward slash as the separator
+  private static final String fileSeparator = "/";
   public HivePartitionLocation(int max, String baseTableLocation, String entireLocation) {
     this.partitionLocation = entireLocation;
     partitionValue = new String[max];
@@ -39,7 +40,6 @@ public class HivePartitionLocation implements PartitionLocation {
     for (int i = 0; i < mostDirs.length; i++) {
       this.partitionValue[i] = mostDirs[i].substring(mostDirs[i].indexOf("=") + 1);
     }
-
   }
   @Override
   public String getPartitionValue(int index) {