----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/400/ -----------------------------------------------------------
(Updated 2011-02-08 11:26:41.125217) Review request for pig and Julien Le Dem. Changes ------- Attaching new patch that addresses the review comments. Summary ------- As production Pig scripts grow longer and longer, Pig Latin has a need to integrate standard programming techniques of separation and code sharing offered by functions and modules. A detailed proposal of adding macro expansion to Pig Latin is posted here: http://wiki.apache.org/pig/TuringCompletePig This addresses bug PIG-1793. https://issues.apache.org/jira/browse/PIG-1793 Diffs (updated) ----- http://svn.apache.org/repos/asf/pig/trunk/build.xml 1068533 http://svn.apache.org/repos/asf/pig/trunk/src/org/apache/pig/Main.java 1068533 http://svn.apache.org/repos/asf/pig/trunk/src/org/apache/pig/parser/AliasMasker.g PRE-CREATION http://svn.apache.org/repos/asf/pig/trunk/src/org/apache/pig/parser/MacroExpansion.g PRE-CREATION http://svn.apache.org/repos/asf/pig/trunk/src/org/apache/pig/parser/MacroImport.g PRE-CREATION http://svn.apache.org/repos/asf/pig/trunk/src/org/apache/pig/parser/ParserUtil.java PRE-CREATION http://svn.apache.org/repos/asf/pig/trunk/src/org/apache/pig/parser/PigMacro.java PRE-CREATION http://svn.apache.org/repos/asf/pig/trunk/src/org/apache/pig/parser/QueryParser.g 1068533 http://svn.apache.org/repos/asf/pig/trunk/src/org/apache/pig/scripting/Pig.java 1068533 http://svn.apache.org/repos/asf/pig/trunk/test/org/apache/pig/test/TestMacroExpansion.java PRE-CREATION Diff: https://reviews.apache.org/r/400/diff Testing ------- Thanks, Richard