svn commit: r1095547 - in /pig/branches/branch-0.9: CHANGES.txt src/org/apache/pig/newplan/logical/expression/CastExpression.java test/org/apache/pig/parser/TestLogicalPlanGenerator.java

[xuefu]

Author: xuefu
Date: Wed Apr 20 22:57:12 2011
New Revision: 1095547

URL: http://svn.apache.org/viewvc?rev=1095547&view=rev
Log:
Regression: Pig gives error <Projection with nothing to reference> for a valid 
query

Modified:
    pig/branches/branch-0.9/CHANGES.txt
    
pig/branches/branch-0.9/src/org/apache/pig/newplan/logical/expression/CastExpression.java
    
pig/branches/branch-0.9/test/org/apache/pig/parser/TestLogicalPlanGenerator.java

Modified: pig/branches/branch-0.9/CHANGES.txt
URL: 
http://svn.apache.org/viewvc/pig/branches/branch-0.9/CHANGES.txt?rev=1095547&r1=1095546&r2=1095547&view=diff
==============================================================================
--- pig/branches/branch-0.9/CHANGES.txt (original)
+++ pig/branches/branch-0.9/CHANGES.txt Wed Apr 20 22:57:12 2011
@@ -32,6 +32,8 @@ PIG-1876: Typed map for Pig (daijy)
 
 IMPROVEMENTS
 
+PIG-2002: Regression: Pig gives error "Projection with nothing to reference!" 
for a valid query (xuefu)
+
 PIG-1921: Improve error messages in new parser (xuefu)
 
 PIG-1996: Pig new parser fails to recognize PARALLEL keywords in a case (xuefu)

Modified: 
pig/branches/branch-0.9/src/org/apache/pig/newplan/logical/expression/CastExpression.java
URL: 
http://svn.apache.org/viewvc/pig/branches/branch-0.9/src/org/apache/pig/newplan/logical/expression/CastExpression.java?rev=1095547&r1=1095546&r2=1095547&view=diff
==============================================================================
--- 
pig/branches/branch-0.9/src/org/apache/pig/newplan/logical/expression/CastExpression.java
 (original)
+++ 
pig/branches/branch-0.9/src/org/apache/pig/newplan/logical/expression/CastExpression.java
 Wed Apr 20 22:57:12 2011
@@ -89,7 +89,7 @@ public class CastExpression extends Unar
         LogicalExpression copy = new CastExpression(
                 lgExpPlan,
                 this.getExpression().deepCopy(lgExpPlan),
-                this.getFieldSchema().deepCopy());
+                castSchema.deepCopy());
         try {
             FuncSpec origFuncSpec = this.getFuncSpec();
             if (origFuncSpec != null ) {

Modified: 
pig/branches/branch-0.9/test/org/apache/pig/parser/TestLogicalPlanGenerator.java
URL: 
http://svn.apache.org/viewvc/pig/branches/branch-0.9/test/org/apache/pig/parser/TestLogicalPlanGenerator.java?rev=1095547&r1=1095546&r2=1095547&view=diff
==============================================================================
--- 
pig/branches/branch-0.9/test/org/apache/pig/parser/TestLogicalPlanGenerator.java
 (original)
+++ 
pig/branches/branch-0.9/test/org/apache/pig/parser/TestLogicalPlanGenerator.java
 Wed Apr 20 22:57:12 2011
@@ -46,6 +46,7 @@ public class TestLogicalPlanGenerator {
                          };
         command = Util.createInputFile("script", "pl", script);
     }
+    
     @Test
     public void test1() {
         String query = "A = load 'x' as ( u:int, v:long, w:bytearray); " + 
@@ -230,9 +231,17 @@ public class TestLogicalPlanGenerator {
     }
 
     @Test
+    public void test19() {
+        String query = "A = load 'x' as (u:map[], v);\n" +
+                       "B = foreach A { T = (chararray)u#'hello'#'world'; 
generate T; };";
+        generateLogicalPlan( query );
+    }
+    
+    @Test
     public void testFilter() {
         String query = "A = load 'x' as ( u:int, v:long, w:bytearray); " + 
-                       "B = filter A by 2 > 1; ";
+                       "B = filter A by 2 > 1;\n" +
+                       "store B into 'y';";
         generateLogicalPlan( query );
     }