Pig gave a ship it? Or do you still need more eyes? 2012/6/18 Julien Le Dem <[email protected]>
> > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/5159/ > ----------------------------------------------------------- > > (Updated June 18, 2012, 6:07 p.m.) > > > Review request for pig. > > > Changes > ------- > > change signature in helper method for consistency > > > Description > ------- > > Pig creates too many instances of Load and Store Funcs. It should be 1 in > the front-end and 1 in the backend > > > This addresses bug PIG-2699. > https://issues.apache.org/jira/browse/PIG-2699 > > > Diffs (updated) > ----- > > /trunk/src/org/apache/pig/PigServer.java 1339793 > > /trunk/src/org/apache/pig/backend/hadoop/executionengine/HExecutionEngine.java > 1339793 > > /trunk/src/org/apache/pig/backend/hadoop/executionengine/mapReduceLayer/MapReduceLauncher.java > 1339793 > > /trunk/src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/relationalOperators/POStore.java > 1339793 > /trunk/src/org/apache/pig/newplan/logical/relational/LOLoad.java 1339793 > /trunk/src/org/apache/pig/newplan/logical/relational/LOStore.java 1339793 > > /trunk/src/org/apache/pig/newplan/logical/relational/LogToPhyTranslationVisitor.java > 1339793 > > /trunk/src/org/apache/pig/newplan/logical/rules/LoadStoreFuncDupSignatureValidator.java > 1339793 > /trunk/src/org/apache/pig/newplan/logical/rules/PartitionFilterOptimizer.java > 1339793 > /trunk/src/org/apache/pig/newplan/logical/rules/TypeCastInserter.java > 1339793 > /trunk/src/org/apache/pig/newplan/logical/visitor/ScalarVisitor.java > 1339793 > /trunk/src/org/apache/pig/parser/FunctionType.java 1339793 > /trunk/src/org/apache/pig/parser/LogicalPlanBuilder.java 1339793 > /trunk/src/org/apache/pig/parser/QueryParserUtils.java 1339793 > /trunk/test/org/apache/pig/TestLoadStoreFuncLifeCycle.java PRE-CREATION > /trunk/test/org/apache/pig/newplan/logical/relational/LOTestHelper.java > PRE-CREATION > /trunk/test/org/apache/pig/parser/TestScalarVisitor.java 1339793 > /trunk/test/org/apache/pig/test/TestInputOutputFileValidator.java 1339793 > /trunk/test/org/apache/pig/test/TestLogToPhyCompiler.java 1339793 > /trunk/test/org/apache/pig/test/TestLogicalPlanBuilder.java 1339793 > /trunk/test/org/apache/pig/test/TestMRCompiler.java 1339793 > /trunk/test/org/apache/pig/test/TestNewPlanFilterRule.java 1339793 > /trunk/test/org/apache/pig/test/TestNewPlanListener.java 1339793 > /trunk/test/org/apache/pig/test/TestNewPlanLogicalOptimizer.java 1339793 > /trunk/test/org/apache/pig/test/TestNewPlanOperatorPlan.java 1339793 > /trunk/test/org/apache/pig/test/TestTypeCheckingValidatorNewLP.java > 1339793 > /trunk/test/org/apache/pig/test/Util.java 1339793 > /trunk/test/org/apache/pig/test/utils/TypeCheckingTestUtil.java 1339793 > > Diff: https://reviews.apache.org/r/5159/diff/ > > > Testing > ------- > > test-commit + new test > > > Thanks, > > Julien Le Dem > >
