You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pig.apache.org by dv...@apache.org on 2011/11/15 05:59:55 UTC

svn commit: r1202044 - in /pig/trunk: CHANGES.txt src/org/apache/pig/builtin/TOP.java

Author: dvryaboy
Date: Tue Nov 15 04:59:54 2011
New Revision: 1202044

URL: http://svn.apache.org/viewvc?rev=1202044&view=rev
Log:
PIG-2365: Current TOP implementation needlessly results in a null bag name

Modified:
    pig/trunk/CHANGES.txt
    pig/trunk/src/org/apache/pig/builtin/TOP.java

Modified: pig/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/pig/trunk/CHANGES.txt?rev=1202044&r1=1202043&r2=1202044&view=diff
==============================================================================
--- pig/trunk/CHANGES.txt (original)
+++ pig/trunk/CHANGES.txt Tue Nov 15 04:59:54 2011
@@ -24,6 +24,8 @@ INCOMPATIBLE CHANGES
 
 IMPROVEMENTS
 
+PIG-2365: Current TOP implementation needlessly results in a null bag name (jcoveney via dvryaboy) 
+
 PIG-2151: Add annotation to specify output schema in Java UDFs (dvryaboy)
 
 PIG-2230: Improved error message for invalid parameter format (xuitingz via olgan)

Modified: pig/trunk/src/org/apache/pig/builtin/TOP.java
URL: http://svn.apache.org/viewvc/pig/trunk/src/org/apache/pig/builtin/TOP.java?rev=1202044&r1=1202043&r2=1202044&view=diff
==============================================================================
--- pig/trunk/src/org/apache/pig/builtin/TOP.java (original)
+++ pig/trunk/src/org/apache/pig/builtin/TOP.java Tue Nov 15 04:59:54 2011
@@ -167,10 +167,7 @@ public class TOP extends EvalFunc<DataBa
             if (input.size() < 3) {
                 return null;
             }
-            Schema.FieldSchema bagFs = new Schema.FieldSchema(null,
-                    input.getField(2).schema, DataType.BAG);
-            return new Schema(bagFs);
-
+            return new Schema(input.getField(2));
         } catch (Exception e) {
             return null;
         }