You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@pig.apache.org by j....@gmail.com on 2012/07/03 23:30:30 UTC

Review Request: PIG-2765: [NEW] Implementing RollupDimensions UDF and adding ROLLUP clause in CUBE operator

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/5733/
-----------------------------------------------------------

Review request for pig and Dmitriy Ryaboy.


Description
-------

This is a review board request for https://issues.apache.org/jira/browse/PIG-2765


This addresses bug PIG-2765.
    https://issues.apache.org/jira/browse/PIG-2765


Diffs
-----

  src/org/apache/pig/builtin/CubeDimensions.java 4f2680f 
  src/org/apache/pig/builtin/RollupDimensions.java PRE-CREATION 
  src/org/apache/pig/newplan/logical/relational/LOCube.java ee60e9b 
  src/org/apache/pig/parser/AliasMasker.g e2166ee 
  src/org/apache/pig/parser/AstPrinter.g b03fd6b 
  src/org/apache/pig/parser/AstValidator.g d5c8229 
  src/org/apache/pig/parser/LogicalPlanBuilder.java a7420a7 
  src/org/apache/pig/parser/LogicalPlanGenerator.g 6c1da32 
  src/org/apache/pig/parser/QueryLexer.g fe68daf 
  src/org/apache/pig/parser/QueryParser.g 719aa94 
  test/org/apache/pig/parser/TestLexer.pig 9505512 
  test/org/apache/pig/parser/TestLogicalPlanGenerator.java 0fb56e2 
  test/org/apache/pig/parser/TestParser.pig 3f62145 
  test/org/apache/pig/parser/TestQueryLexer.java 921045f 
  test/org/apache/pig/parser/TestQueryParser.java 6e4805d 
  test/org/apache/pig/test/TestCubeOperator.java db14976 
  test/org/apache/pig/test/TestRollupDimensions.java PRE-CREATION 

Diff: https://reviews.apache.org/r/5733/diff/


Testing
-------

Unit tests: All passed

Pre-commit tests: All passed
ant clean test-commit


Thanks,

Prasanth_J