You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pig.apache.org by ch...@apache.org on 2013/08/16 21:05:39 UTC

svn commit: r1514853 - in /pig/trunk: CHANGES.txt contrib/piggybank/java/src/main/java/org/apache/pig/piggybank/evaluation/util/Top.java src/docs/src/documentation/content/xdocs/func.xml src/org/apache/pig/builtin/TOP.java

Author: cheolsoo
Date: Fri Aug 16 19:05:38 2013
New Revision: 1514853

URL: http://svn.apache.org/r1514853
Log:
PIG-3405: Top UDF documentation indicates improper use (aniket486 via cheolsoo)

Modified:
    pig/trunk/CHANGES.txt
    pig/trunk/contrib/piggybank/java/src/main/java/org/apache/pig/piggybank/evaluation/util/Top.java
    pig/trunk/src/docs/src/documentation/content/xdocs/func.xml
    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=1514853&r1=1514852&r2=1514853&view=diff
==============================================================================
--- pig/trunk/CHANGES.txt (original)
+++ pig/trunk/CHANGES.txt Fri Aug 16 19:05:38 2013
@@ -212,6 +212,8 @@ PIG-3013: BinInterSedes improve chararra
 
 BUG FIXES
 
+PIG-3405: Top UDF documentation indicates improper use (aniket486 via cheolsoo)
+
 PIG-3425: Hive jdo api jar referenced in pig script throws error (deepesh via cheolsoo)
 
 PIG-3422: AvroStorage failed to read paths separated by commas (yuanlid via rohini)

Modified: pig/trunk/contrib/piggybank/java/src/main/java/org/apache/pig/piggybank/evaluation/util/Top.java
URL: http://svn.apache.org/viewvc/pig/trunk/contrib/piggybank/java/src/main/java/org/apache/pig/piggybank/evaluation/util/Top.java?rev=1514853&r1=1514852&r2=1514853&view=diff
==============================================================================
--- pig/trunk/contrib/piggybank/java/src/main/java/org/apache/pig/piggybank/evaluation/util/Top.java (original)
+++ pig/trunk/contrib/piggybank/java/src/main/java/org/apache/pig/piggybank/evaluation/util/Top.java Fri Aug 16 19:05:38 2013
@@ -58,7 +58,7 @@ import org.apache.pig.impl.logicalLayer.
  * C = FOREACH B generate FLATTEN(group), COUNT(*) as count;
  * D = GROUP C BY first; // again group by first 
  * topResults = FOREACH D { 
- *          result = Top(10, 2, C); // and retain top 10 occurrences of 'second' in first 
+ *          result = Top(10, 1, C); // and retain top 10 occurrences of 'second' in first 
  *          GENERATE FLATTEN(result); 
  *  }
  */

Modified: pig/trunk/src/docs/src/documentation/content/xdocs/func.xml
URL: http://svn.apache.org/viewvc/pig/trunk/src/docs/src/documentation/content/xdocs/func.xml?rev=1514853&r1=1514852&r2=1514853&view=diff
==============================================================================
--- pig/trunk/src/docs/src/documentation/content/xdocs/func.xml (original)
+++ pig/trunk/src/docs/src/documentation/content/xdocs/func.xml Fri Aug 16 19:05:38 2013
@@ -5224,7 +5224,7 @@ B = GROUP A BY (first, second);
 C = FOREACH B generate FLATTEN(group), COUNT(A) as count;
 D = GROUP C BY first; // again group by first
 topResults = FOREACH D {
-    result = TOP(10, 2, C); // and retain top 10 occurrences of 'second' in first
+    result = TOP(10, 1, C); // and retain top 10 occurrences of 'second' in first
     GENERATE FLATTEN(result);
 }
 </source>

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=1514853&r1=1514852&r2=1514853&view=diff
==============================================================================
--- pig/trunk/src/org/apache/pig/builtin/TOP.java (original)
+++ pig/trunk/src/org/apache/pig/builtin/TOP.java Fri Aug 16 19:05:38 2013
@@ -58,7 +58,7 @@ import org.apache.pig.impl.logicalLayer.
  * C = FOREACH B generate FLATTEN(group), COUNT(*) as count;
  * D = GROUP C BY first; // again group by first 
  * topResults = FOREACH D { 
- *          result = Top(10, 2, C); // and retain top 10 occurrences of 'second' in first 
+ *          result = Top(10, 1, C); // and retain top 10 occurrences of 'second' in first 
  *          GENERATE FLATTEN(result); 
  *  }
  */