You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pig.apache.org by jc...@apache.org on 2012/08/09 18:58:56 UTC

svn commit: r1371315 - in /pig/trunk: CHANGES.txt contrib/piggybank/java/src/test/java/org/apache/pig/piggybank/test/storage/avro/TestAvroStorageUtils.java

Author: jcoveney
Date: Thu Aug  9 16:58:55 2012
New Revision: 1371315

URL: http://svn.apache.org/viewvc?rev=1371315&view=rev
Log:
PIG-2860: TestAvroStorageUtils.testGetConcretePathFromGlob fails on some version of hadoop (cheolsoo via jcoveney)

Modified:
    pig/trunk/CHANGES.txt
    pig/trunk/contrib/piggybank/java/src/test/java/org/apache/pig/piggybank/test/storage/avro/TestAvroStorageUtils.java

Modified: pig/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/pig/trunk/CHANGES.txt?rev=1371315&r1=1371314&r2=1371315&view=diff
==============================================================================
--- pig/trunk/CHANGES.txt (original)
+++ pig/trunk/CHANGES.txt Thu Aug  9 16:58:55 2012
@@ -230,6 +230,8 @@ OPTIMIZATIONS
 
 BUG FIXES
 
+PIG-2860: TestAvroStorageUtils.testGetConcretePathFromGlob fails on some version of hadoop (cheolsoo via jcoveney)
+
 PIG-2861: PlanHelper imports org.python.google.common.collect.Lists instead of org.google.common.collect.Lists (jcoveney)
 
 PIG-2849: Errors in document Getting Started (miyakawataku via billgraham)

Modified: pig/trunk/contrib/piggybank/java/src/test/java/org/apache/pig/piggybank/test/storage/avro/TestAvroStorageUtils.java
URL: http://svn.apache.org/viewvc/pig/trunk/contrib/piggybank/java/src/test/java/org/apache/pig/piggybank/test/storage/avro/TestAvroStorageUtils.java?rev=1371315&r1=1371314&r2=1371315&view=diff
==============================================================================
--- pig/trunk/contrib/piggybank/java/src/test/java/org/apache/pig/piggybank/test/storage/avro/TestAvroStorageUtils.java (original)
+++ pig/trunk/contrib/piggybank/java/src/test/java/org/apache/pig/piggybank/test/storage/avro/TestAvroStorageUtils.java Thu Aug  9 16:58:55 2012
@@ -128,8 +128,7 @@ public class TestAvroStorageUtils {
 
     @Test
     public void testGetConcretePathFromGlob() throws IOException {
-        final String defaultscheme = "file:";
-        final String basedir = System.getProperty("user.dir");
+        final String basedir = "file://" + System.getProperty("user.dir");
         final String tempdir = Long.toString(System.currentTimeMillis());
         final String nonexistentpath = basedir + "/" + tempdir + "/this_path_does_not_exist";
 
@@ -140,7 +139,7 @@ public class TestAvroStorageUtils {
         // existent path
         String locationStr = basedir;
         concretePath = AvroStorageUtils.getConcretePathFromGlob(locationStr, job);
-        assertEquals(defaultscheme + basedir, concretePath.toString());
+        assertEquals(basedir, concretePath.toUri().toString());
 
         // non-existent path
         locationStr = nonexistentpath;