You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pig.apache.org by ya...@apache.org on 2010/06/24 03:10:50 UTC

svn commit: r957399 - in /hadoop/pig/trunk/contrib/zebra: ./ src/test/org/apache/hadoop/zebra/pig/

Author: yanz
Date: Thu Jun 24 01:10:49 2010
New Revision: 957399

URL: http://svn.apache.org/viewvc?rev=957399&view=rev
Log:
PIG-1453 Intermittent failure in pigtest (yanz)

Modified:
    hadoop/pig/trunk/contrib/zebra/CHANGES.txt
    hadoop/pig/trunk/contrib/zebra/src/test/org/apache/hadoop/zebra/pig/TestCogroup.java
    hadoop/pig/trunk/contrib/zebra/src/test/org/apache/hadoop/zebra/pig/TestMergeJoinPrune.java
    hadoop/pig/trunk/contrib/zebra/src/test/org/apache/hadoop/zebra/pig/TestOrderPreserveUnion.java
    hadoop/pig/trunk/contrib/zebra/src/test/org/apache/hadoop/zebra/pig/TestOrderPreserveUnionHDFS.java
    hadoop/pig/trunk/contrib/zebra/src/test/org/apache/hadoop/zebra/pig/TestSortedTableUnion.java
    hadoop/pig/trunk/contrib/zebra/src/test/org/apache/hadoop/zebra/pig/TestSortedTableUnionMergeJoin.java
    hadoop/pig/trunk/contrib/zebra/src/test/org/apache/hadoop/zebra/pig/TestSortedTableUnionSourceTableProj.java
    hadoop/pig/trunk/contrib/zebra/src/test/org/apache/hadoop/zebra/pig/TestTableMergeJoin.java
    hadoop/pig/trunk/contrib/zebra/src/test/org/apache/hadoop/zebra/pig/TestTableMergeJoinAfterFilter.java
    hadoop/pig/trunk/contrib/zebra/src/test/org/apache/hadoop/zebra/pig/TestTableMergeJoinMultipleColsSort.java
    hadoop/pig/trunk/contrib/zebra/src/test/org/apache/hadoop/zebra/pig/TestTableSortStorer.java
    hadoop/pig/trunk/contrib/zebra/src/test/org/apache/hadoop/zebra/pig/TestTableSortStorerDesc.java

Modified: hadoop/pig/trunk/contrib/zebra/CHANGES.txt
URL: http://svn.apache.org/viewvc/hadoop/pig/trunk/contrib/zebra/CHANGES.txt?rev=957399&r1=957398&r2=957399&view=diff
==============================================================================
--- hadoop/pig/trunk/contrib/zebra/CHANGES.txt (original)
+++ hadoop/pig/trunk/contrib/zebra/CHANGES.txt Thu Jun 24 01:10:49 2010
@@ -90,6 +90,8 @@ Trunk (unreleased changes)
 
   BUG FIXES
 
+    PIG-1453 Intermittent failure in pigtest (yanz)
+
     PIG-1432 There are some debuging info output to STDOUT in PIG's TableStorer call path (yanz)
 
     PIG-1421 Name node calls made by each mapper (xuefuz via yanz)

Modified: hadoop/pig/trunk/contrib/zebra/src/test/org/apache/hadoop/zebra/pig/TestCogroup.java
URL: http://svn.apache.org/viewvc/hadoop/pig/trunk/contrib/zebra/src/test/org/apache/hadoop/zebra/pig/TestCogroup.java?rev=957399&r1=957398&r2=957399&view=diff
==============================================================================
--- hadoop/pig/trunk/contrib/zebra/src/test/org/apache/hadoop/zebra/pig/TestCogroup.java (original)
+++ hadoop/pig/trunk/contrib/zebra/src/test/org/apache/hadoop/zebra/pig/TestCogroup.java Thu Jun 24 01:10:49 2010
@@ -105,6 +105,7 @@ public class TestCogroup extends BaseTes
   @AfterClass
   public static void tearDown() throws Exception {
     pigServer.shutdown();
+    removeDir(pathTable);
   }
 
   /**
@@ -167,9 +168,9 @@ public class TestCogroup extends BaseTes
     String foreach = "records6 = foreach records5 generate flatten(records3), flatten(records4);";
     pigServer.registerQuery(foreach);
 
-    Path newPath = new Path("testStorer");
+    Path newPath = new Path("TestCogroup.testStorer");
     System.out.println("newPath = " + newPath);
-    removeDir(newPath);
+    removeDir(getTableFullPath(newPath.toString()+"1"));
 
     /*
      * Table1 creation
@@ -182,6 +183,6 @@ public class TestCogroup extends BaseTes
                 + "('[records3::SF_a]; [records4::SF_a]')");
     Assert.assertNull(pigJob.getException());
     
-    removeDir(newPath);
+    removeDir(getTableFullPath(newPath.toString()+"1"));
   }
 }

Modified: hadoop/pig/trunk/contrib/zebra/src/test/org/apache/hadoop/zebra/pig/TestMergeJoinPrune.java
URL: http://svn.apache.org/viewvc/hadoop/pig/trunk/contrib/zebra/src/test/org/apache/hadoop/zebra/pig/TestMergeJoinPrune.java?rev=957399&r1=957398&r2=957399&view=diff
==============================================================================
--- hadoop/pig/trunk/contrib/zebra/src/test/org/apache/hadoop/zebra/pig/TestMergeJoinPrune.java (original)
+++ hadoop/pig/trunk/contrib/zebra/src/test/org/apache/hadoop/zebra/pig/TestMergeJoinPrune.java Thu Jun 24 01:10:49 2010
@@ -392,7 +392,7 @@ public class TestMergeJoinPrune extends 
     t1++;
     
     String table1path = pathTable1.toString() + Integer.toString(t1);
-     
+    removeDir(new Path(table1path));
 
 ExecJob pigJob =pigServer.store("sort1", table1path, TableStorer.class.getCanonicalName()
         + "('[a, b, c]; [d, e, f, r1, m1]')");
@@ -415,6 +415,7 @@ Assert.assertNull(pigJob.getException())
      */
     this.t1++;
     String table2path = this.pathTable2.toString() + Integer.toString(this.t1);
+    removeDir(new Path(table2path));
     pigJob = pigServer.store("sort2", table2path, TableStorer.class.getCanonicalName()
         + "('[a, b, c]; [d,e,f,r1,m1]')");
     if (pigJob.getException() != null){

Modified: hadoop/pig/trunk/contrib/zebra/src/test/org/apache/hadoop/zebra/pig/TestOrderPreserveUnion.java
URL: http://svn.apache.org/viewvc/hadoop/pig/trunk/contrib/zebra/src/test/org/apache/hadoop/zebra/pig/TestOrderPreserveUnion.java?rev=957399&r1=957398&r2=957399&view=diff
==============================================================================
--- hadoop/pig/trunk/contrib/zebra/src/test/org/apache/hadoop/zebra/pig/TestOrderPreserveUnion.java (original)
+++ hadoop/pig/trunk/contrib/zebra/src/test/org/apache/hadoop/zebra/pig/TestOrderPreserveUnion.java Thu Jun 24 01:10:49 2010
@@ -21,6 +21,8 @@ package org.apache.hadoop.zebra.pig;
 import java.io.ByteArrayOutputStream;
 import java.io.IOException;
 import java.io.PrintWriter;
+import java.util.ArrayList;
+import java.util.HashMap;
 import java.util.Iterator;
 import java.util.StringTokenizer;
 
@@ -90,6 +92,7 @@ public class TestOrderPreserveUnion exte
   @AfterClass
   public static void tearDown() throws Exception {
     pigServer.shutdown();
+    removeDir(pathTable);
   }
 
   /**
@@ -153,12 +156,14 @@ public class TestOrderPreserveUnion exte
     /*
      * Table1 creation
      */
+    removeDir(getTableFullPath(newPath.toString()+"1"));
     pigServer
         .store(
             "srecs",
             newPath.toString()+"1",
             TableStorer.class.getCanonicalName()
                 + "('[SF_a, SF_b, SF_c]; [SF_e]')");
+    removeDir(getTableFullPath(newPath.toString()+"2"));
     pigServer
         .store(
             "srecs",
@@ -173,26 +178,37 @@ public class TestOrderPreserveUnion exte
         + "' USING org.apache.hadoop.zebra.pig.TableLoader('SF_a, source_table, SF_b', 'sorted');";
     pigServer.registerQuery(query4);
 
-    // check JOIN content
-    Iterator<Tuple> it3 = pigServer.openIterator("records2");
-    int row = 0, index, tindex = -1;
-    Tuple RowValue3 = null;
-    while (it3.hasNext()) {
-      // Last row value
-      RowValue3 = it3.next();
-      Assert.assertEquals(3, RowValue3.size());
-      index = row;
-      if (index > 9)
-        index -= 10;
-      row++;
-      if (row == 1)
-        tindex = (Integer) RowValue3.get(1);
-      else if (row == 11)
-        tindex = 1 - tindex;
-      Assert.assertEquals(index+"_00", RowValue3.get(0));
-      Assert.assertEquals(index+"_01", RowValue3.get(2));
-      Assert.assertEquals(tindex, RowValue3.get(1));
-    }
+    HashMap<Integer, ArrayList<ArrayList<Object>>> resultTable
+    = new HashMap<Integer, ArrayList<ArrayList<Object>>>();
+
+    ArrayList<ArrayList<Object>> rows = new ArrayList<ArrayList<Object>>();
+    
+    for (int i = 0; i < 10; i++)
+      addResultRow(rows, i+"_00", 0, i+"_01");
+    resultTable.put(0, rows);
+    
+    rows = new ArrayList<ArrayList<Object>>();
+    for (int i = 0; i < 10; i++)
+      addResultRow(rows, i+"_00", 1,  i+"_01");
+    resultTable.put(1, rows);
+
+ // Verify union table
+    Iterator<Tuple> it = pigServer.openIterator("records2");
+    int row = verifyTable(resultTable, 0, 1, it);
+    
     Assert.assertEquals(20, row);
   }
+  
+  /**
+   *Add a row to expected results table
+   * 
+   */
+  private void addResultRow(ArrayList<ArrayList<Object>> resultTable, Object ... values) {
+    ArrayList<Object> resultRow = new ArrayList<Object>();
+    
+    for (int i = 0; i < values.length; i++) {
+      resultRow.add(values[i]);
+    }
+    resultTable.add(resultRow);
+  }
 } 

Modified: hadoop/pig/trunk/contrib/zebra/src/test/org/apache/hadoop/zebra/pig/TestOrderPreserveUnionHDFS.java
URL: http://svn.apache.org/viewvc/hadoop/pig/trunk/contrib/zebra/src/test/org/apache/hadoop/zebra/pig/TestOrderPreserveUnionHDFS.java?rev=957399&r1=957398&r2=957399&view=diff
==============================================================================
--- hadoop/pig/trunk/contrib/zebra/src/test/org/apache/hadoop/zebra/pig/TestOrderPreserveUnionHDFS.java (original)
+++ hadoop/pig/trunk/contrib/zebra/src/test/org/apache/hadoop/zebra/pig/TestOrderPreserveUnionHDFS.java Thu Jun 24 01:10:49 2010
@@ -19,6 +19,7 @@
 package org.apache.hadoop.zebra.pig;
 
 import java.io.IOException;
+import java.util.HashMap;
 import java.util.Iterator;
 import java.util.ArrayList;
 
@@ -137,12 +138,14 @@ public class TestOrderPreserveUnionHDFS 
 		
 		// Store sorted tables
 		++fileId;  // increment filename suffix
-		String pathSort1 = pathTable1.toString() + Integer.toString(fileId);
+		String pathSort1 = "TestOrderPerserveSimple1" + Integer.toString(fileId);
+	  removeDir(getTableFullPath(pathSort1));
 		pigJob = pigServer.store("sort1", pathSort1, TableStorer.class.getCanonicalName() +
 			"('[a, b]; [c]')");
 		Assert.assertNull(pigJob.getException());
 		
-		String pathSort2 = pathTable2.toString() + Integer.toString(fileId);
+		String pathSort2 = "TestOrderPerserveSimple2" + Integer.toString(fileId);
+	  removeDir(getTableFullPath(pathSort2));
 		pigJob = pigServer.store("sort2", pathSort2, TableStorer.class.getCanonicalName() +
 			"('[a, b]; [c]')");
 		Assert.assertNull(pigJob.getException());
@@ -150,109 +153,37 @@ public class TestOrderPreserveUnionHDFS 
 		String queryLoad = "records1 = LOAD '"
 	        + pathSort1 + ","
 	        + pathSort2
-	        +	"' USING org.apache.hadoop.zebra.pig.TableLoader('a,b,c', 'sorted');";
+	        +	"' USING org.apache.hadoop.zebra.pig.TableLoader('a,b,c, source_table', 'sorted');";
 		
 		System.out.println("queryLoad: " + queryLoad);
 		
 		pigServer.registerQuery(queryLoad);
 		
 		// Verify union table
-		ArrayList<ArrayList<Object>> resultTable = new ArrayList<ArrayList<Object>>();
-		
-		addResultRow(resultTable, "a1",	"z",	"5");
-		addResultRow(resultTable, "a2",	"r",	"4");
-		addResultRow(resultTable, "a3",	"e",	"3");
-		addResultRow(resultTable, "a4",	"a",	"1");
-		
-		addResultRow(resultTable, "b1",	"a",	"a");
-		addResultRow(resultTable, "b2",	"a",	"a");
-		addResultRow(resultTable, "b3",	"a",	"a");
-		addResultRow(resultTable, "b4",	"a",	"a");
+		HashMap<Integer, ArrayList<ArrayList<Object>>> resultTable
+    = new HashMap<Integer, ArrayList<ArrayList<Object>>>();
+		ArrayList<ArrayList<Object>> rows = new ArrayList<ArrayList<Object>>();
+		addResultRow(rows, "a1",	"z",	"5", 0);
+		addResultRow(rows, "a2",	"r",	"4", 0);
+		addResultRow(rows, "a3",	"e",	"3", 0);
+		addResultRow(rows, "a4",	"a",	"1", 0);
+		resultTable.put(0, rows);
+		
+		rows = new ArrayList<ArrayList<Object>>();
+		addResultRow(rows, "b1",	"a",	"a", 1);
+		addResultRow(rows, "b2",	"a",	"a", 1);
+		addResultRow(rows, "b3",	"a",	"a", 1);
+		addResultRow(rows, "b4",	"a",	"a", 1);
+		resultTable.put(1, rows);
 		
 		// Verify union table
 		Iterator<Tuple> it = pigServer.openIterator("records1");
-		int numbRows = verifyTable(resultTable, 0, it);
+		int numbRows = verifyTable(resultTable, 0, 3, it);
 		
 		Assert.assertEquals(numbRows, table1.length + table2.length);
 	}
 	
 	/**
-	 * Verify union output table with expected results
-	 * 
-	 */
-	private int verifyTable(ArrayList<ArrayList<Object>> resultTable, int keyColumn, Iterator<Tuple> it) throws IOException {
-		int numbRows = 0;
-		int index = 0;
-		Object value = resultTable.get(index).get(keyColumn);  // get value of primary key
-		
-		while (it.hasNext()) {
-			Tuple rowValues = it.next();
-			
-			// If last primary sort key does match then search for next matching key
-			if (! compareObj(value, rowValues.get(keyColumn))) {
-				int subIndex = index + 1;
-				while (subIndex < resultTable.size()) {
-					if ( ! compareObj(value, resultTable.get(subIndex).get(keyColumn)) ) {  // found new key
-						index = subIndex;
-						value = resultTable.get(index).get(keyColumn);
-						break;
-					}
-					++subIndex;
-				}
-				Assert.assertEquals("Table comparison error for row : " + numbRows + " - no key found for : "
-					+ rowValues.get(keyColumn), value, rowValues.get(keyColumn));
-			}
-			// Search for matching row with this primary key
-			int subIndex = index;
-			
-			while (subIndex < resultTable.size()) {
-				// Compare row
-				ArrayList<Object> resultRow = resultTable.get(subIndex);
-				if ( compareRow(rowValues, resultRow) )
-					break; // found matching row
-				++subIndex;
-				Assert.assertEquals("Table comparison error for row : " + numbRows + " - no matching row found for : "
-					+ rowValues.get(keyColumn), value, resultTable.get(subIndex).get(keyColumn));
-			}
-			++numbRows;
-		}
-		Assert.assertEquals(resultTable.size(), numbRows);  // verify expected row count
-		return numbRows;
-	}
-	
-	/**
-	 * Compare table rows
-	 * 
-	 */
-	private boolean compareRow(Tuple rowValues, ArrayList<Object> resultRow) throws IOException {
-		boolean result = true;
-		Assert.assertEquals(resultRow.size(), rowValues.size());
-		for (int i = 0; i < rowValues.size(); ++i) {
-			if (! compareObj(rowValues.get(i), resultRow.get(i)) ) {
-				result = false;
-				break;
-			}
-		}
-		return result;
-	}
-	
-	/**
-	 * Compare table values
-	 * 
-	 */
-	private boolean compareObj(Object object1, Object object2) {
-		if (object1 == null) {
-			if (object2 == null)
-				return true;
-			else
-				return false;
-		} else if (object1.equals(object2))
-			return true;
-		else
-			return false;
-	}
-	
-	/**
 	 *Add a row to expected results table
 	 * 
 	 */

Modified: hadoop/pig/trunk/contrib/zebra/src/test/org/apache/hadoop/zebra/pig/TestSortedTableUnion.java
URL: http://svn.apache.org/viewvc/hadoop/pig/trunk/contrib/zebra/src/test/org/apache/hadoop/zebra/pig/TestSortedTableUnion.java?rev=957399&r1=957398&r2=957399&view=diff
==============================================================================
--- hadoop/pig/trunk/contrib/zebra/src/test/org/apache/hadoop/zebra/pig/TestSortedTableUnion.java (original)
+++ hadoop/pig/trunk/contrib/zebra/src/test/org/apache/hadoop/zebra/pig/TestSortedTableUnion.java Thu Jun 24 01:10:49 2010
@@ -153,6 +153,8 @@ public class TestSortedTableUnion extend
      */
     Path newPath = new Path(getCurrentMethodName());
 
+    removeDir(getTableFullPath(newPath.toString()+"1"));
+    removeDir(getTableFullPath(newPath.toString()+"2"));
     /*
      * Table1 creation
      */

Modified: hadoop/pig/trunk/contrib/zebra/src/test/org/apache/hadoop/zebra/pig/TestSortedTableUnionMergeJoin.java
URL: http://svn.apache.org/viewvc/hadoop/pig/trunk/contrib/zebra/src/test/org/apache/hadoop/zebra/pig/TestSortedTableUnionMergeJoin.java?rev=957399&r1=957398&r2=957399&view=diff
==============================================================================
--- hadoop/pig/trunk/contrib/zebra/src/test/org/apache/hadoop/zebra/pig/TestSortedTableUnionMergeJoin.java (original)
+++ hadoop/pig/trunk/contrib/zebra/src/test/org/apache/hadoop/zebra/pig/TestSortedTableUnionMergeJoin.java Thu Jun 24 01:10:49 2010
@@ -163,6 +163,9 @@ public class TestSortedTableUnionMergeJo
      */
     Path newPath = new Path(getCurrentMethodName());
 
+    removeDir(getTableFullPath(newPath.toString()+"1"));
+    removeDir(getTableFullPath(newPath.toString()+"2"));
+    
     /*
      * Table1 creation
      */

Modified: hadoop/pig/trunk/contrib/zebra/src/test/org/apache/hadoop/zebra/pig/TestSortedTableUnionSourceTableProj.java
URL: http://svn.apache.org/viewvc/hadoop/pig/trunk/contrib/zebra/src/test/org/apache/hadoop/zebra/pig/TestSortedTableUnionSourceTableProj.java?rev=957399&r1=957398&r2=957399&view=diff
==============================================================================
--- hadoop/pig/trunk/contrib/zebra/src/test/org/apache/hadoop/zebra/pig/TestSortedTableUnionSourceTableProj.java (original)
+++ hadoop/pig/trunk/contrib/zebra/src/test/org/apache/hadoop/zebra/pig/TestSortedTableUnionSourceTableProj.java Thu Jun 24 01:10:49 2010
@@ -166,12 +166,14 @@ public class TestSortedTableUnionSourceT
     /*
      * Table1 creation
      */
+    removeDir(getTableFullPath(newPath.toString()+"1"));
     pigServer
         .store(
             "srecs",
             newPath.toString()+"1",
             TableStorer.class.getCanonicalName()
                 + "('[SF_a, SF_b, SF_c]; [SF_e]')");
+    removeDir(getTableFullPath(newPath.toString()+"2"));
     pigServer
         .store(
             "srecs",

Modified: hadoop/pig/trunk/contrib/zebra/src/test/org/apache/hadoop/zebra/pig/TestTableMergeJoin.java
URL: http://svn.apache.org/viewvc/hadoop/pig/trunk/contrib/zebra/src/test/org/apache/hadoop/zebra/pig/TestTableMergeJoin.java?rev=957399&r1=957398&r2=957399&view=diff
==============================================================================
--- hadoop/pig/trunk/contrib/zebra/src/test/org/apache/hadoop/zebra/pig/TestTableMergeJoin.java (original)
+++ hadoop/pig/trunk/contrib/zebra/src/test/org/apache/hadoop/zebra/pig/TestTableMergeJoin.java Thu Jun 24 01:10:49 2010
@@ -163,7 +163,7 @@ public class TestTableMergeJoin extends 
      * "[SF_a, SF_b, SF_c]; [SF_e, SF_f, SF_g]", false, conf);
      */
     Path newPath = new Path(getCurrentMethodName());
-
+    removeDir(getTableFullPath(newPath.toString()+"1"));
     /*
      * Table1 creation
      */
@@ -195,7 +195,7 @@ public class TestTableMergeJoin extends 
     }
     Assert.assertEquals(10, row);
     */
-    
+    removeDir(getTableFullPath(newPath.toString()+"2"));
     /*
      * Table2 creation
      */

Modified: hadoop/pig/trunk/contrib/zebra/src/test/org/apache/hadoop/zebra/pig/TestTableMergeJoinAfterFilter.java
URL: http://svn.apache.org/viewvc/hadoop/pig/trunk/contrib/zebra/src/test/org/apache/hadoop/zebra/pig/TestTableMergeJoinAfterFilter.java?rev=957399&r1=957398&r2=957399&view=diff
==============================================================================
--- hadoop/pig/trunk/contrib/zebra/src/test/org/apache/hadoop/zebra/pig/TestTableMergeJoinAfterFilter.java (original)
+++ hadoop/pig/trunk/contrib/zebra/src/test/org/apache/hadoop/zebra/pig/TestTableMergeJoinAfterFilter.java Thu Jun 24 01:10:49 2010
@@ -167,6 +167,7 @@ public class TestTableMergeJoinAfterFilt
     /*
      * Table1 creation
      */
+    removeDir(getTableFullPath(newPath.toString()+"1"));
     pigServer
         .store(
             "srecs",
@@ -182,6 +183,7 @@ public class TestTableMergeJoinAfterFilt
     /*
      * Table2 creation
      */
+    removeDir(getTableFullPath(newPath.toString()+"2"));
     pigServer
         .store(
             "srecs",

Modified: hadoop/pig/trunk/contrib/zebra/src/test/org/apache/hadoop/zebra/pig/TestTableMergeJoinMultipleColsSort.java
URL: http://svn.apache.org/viewvc/hadoop/pig/trunk/contrib/zebra/src/test/org/apache/hadoop/zebra/pig/TestTableMergeJoinMultipleColsSort.java?rev=957399&r1=957398&r2=957399&view=diff
==============================================================================
--- hadoop/pig/trunk/contrib/zebra/src/test/org/apache/hadoop/zebra/pig/TestTableMergeJoinMultipleColsSort.java (original)
+++ hadoop/pig/trunk/contrib/zebra/src/test/org/apache/hadoop/zebra/pig/TestTableMergeJoinMultipleColsSort.java Thu Jun 24 01:10:49 2010
@@ -156,7 +156,7 @@ public class TestTableMergeJoinMultipleC
      * Table1 creation
      */
 
-    
+    removeDir(getTableFullPath(newPath.toString()+"1"));    
     pigServer
         .store(
             "srecs",
@@ -171,7 +171,7 @@ public class TestTableMergeJoinMultipleC
     /*
      * Table2 creation
      */
-
+    removeDir(getTableFullPath(newPath.toString()+"2"));
     pigServer
         .store(
             "srecs",

Modified: hadoop/pig/trunk/contrib/zebra/src/test/org/apache/hadoop/zebra/pig/TestTableSortStorer.java
URL: http://svn.apache.org/viewvc/hadoop/pig/trunk/contrib/zebra/src/test/org/apache/hadoop/zebra/pig/TestTableSortStorer.java?rev=957399&r1=957398&r2=957399&view=diff
==============================================================================
--- hadoop/pig/trunk/contrib/zebra/src/test/org/apache/hadoop/zebra/pig/TestTableSortStorer.java (original)
+++ hadoop/pig/trunk/contrib/zebra/src/test/org/apache/hadoop/zebra/pig/TestTableSortStorer.java Thu Jun 24 01:10:49 2010
@@ -159,8 +159,8 @@ public class TestTableSortStorer extends
      * BasicTable.Writer(pathTable, "SF_a,SF_b,SF_c,SF_d,SF_e,SF_f,SF_g",
      * "[SF_a, SF_b, SF_c]; [SF_e, SF_f, SF_g]", false, conf);
      */
-    Path newPath = new Path(getCurrentMethodName());
-
+    Path newPath = getTableFullPath(getCurrentMethodName());
+    removeDir(newPath);
     pigServer
         .store(
             "srecs",

Modified: hadoop/pig/trunk/contrib/zebra/src/test/org/apache/hadoop/zebra/pig/TestTableSortStorerDesc.java
URL: http://svn.apache.org/viewvc/hadoop/pig/trunk/contrib/zebra/src/test/org/apache/hadoop/zebra/pig/TestTableSortStorerDesc.java?rev=957399&r1=957398&r2=957399&view=diff
==============================================================================
--- hadoop/pig/trunk/contrib/zebra/src/test/org/apache/hadoop/zebra/pig/TestTableSortStorerDesc.java (original)
+++ hadoop/pig/trunk/contrib/zebra/src/test/org/apache/hadoop/zebra/pig/TestTableSortStorerDesc.java Thu Jun 24 01:10:49 2010
@@ -159,8 +159,8 @@ public class TestTableSortStorerDesc ext
      * BasicTable.Writer(pathTable, "SF_a,SF_b,SF_c,SF_d,SF_e,SF_f,SF_g",
      * "[SF_a, SF_b, SF_c]; [SF_e, SF_f, SF_g]", false, conf);
      */
-    Path newPath = new Path(getCurrentMethodName());
-
+    Path newPath = getTableFullPath(getCurrentMethodName());
+    removeDir(newPath);
     pigServer
         .store(
             "srecs",