pig-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From da...@apache.org
Subject svn commit: r1359814 - in /pig/trunk: CHANGES.txt test/org/apache/pig/test/TestUDFContext.java
Date Tue, 10 Jul 2012 18:14:47 GMT
Author: daijy
Date: Tue Jul 10 18:14:46 2012
New Revision: 1359814

URL: http://svn.apache.org/viewvc?rev=1359814&view=rev
Log:
PIG-2809: TestUDFContext broken by PIG-2699

Modified:
    pig/trunk/CHANGES.txt
    pig/trunk/test/org/apache/pig/test/TestUDFContext.java

Modified: pig/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/pig/trunk/CHANGES.txt?rev=1359814&r1=1359813&r2=1359814&view=diff
==============================================================================
--- pig/trunk/CHANGES.txt (original)
+++ pig/trunk/CHANGES.txt Tue Jul 10 18:14:46 2012
@@ -192,6 +192,8 @@ OPTIMIZATIONS
 
 BUG FIXES
 
+PIG-2809: TestUDFContext broken by PIG-2699 (julienledem via daijy)
+
 PIG-2807: TestParser TestPigStorage TestNewPlanOperatorPlan broken by PIG-2699 (julienledem
via daijy)
 
 PIG-2782: Specifying sorting field(s) at nightly.conf (cheolsoo via daijy)

Modified: pig/trunk/test/org/apache/pig/test/TestUDFContext.java
URL: http://svn.apache.org/viewvc/pig/trunk/test/org/apache/pig/test/TestUDFContext.java?rev=1359814&r1=1359813&r2=1359814&view=diff
==============================================================================
--- pig/trunk/test/org/apache/pig/test/TestUDFContext.java (original)
+++ pig/trunk/test/org/apache/pig/test/TestUDFContext.java Tue Jul 10 18:14:46 2012
@@ -97,7 +97,9 @@ public class TestUDFContext {
 
         // required fields property should be set because f results does not
         // require the third column c, so properties should not be null
-        assertNotNull(props.get("l"));
+        assertTrue(
+                "properties in udf context for load should not be empty: "+props, 
+                props.keySet().size()>0);
         
         // the new statement for alias f below will require all columns,
         // so this time required fields property for loader should not be set
@@ -105,8 +107,9 @@ public class TestUDFContext {
         pig.explain("f", System.out);
         props = UDFContext.getUDFContext().getUDFProperties(PigStorage.class);
 
-        assertTrue("properties in udf context for load should be null", 
-                props.get("l") == null);
+        assertTrue(
+                "properties in udf context for load should be empty: "+props, 
+                props.keySet().size() == 0);
 
         
     }



Mime
View raw message