-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/5159/
-----------------------------------------------------------
Review request for pig.
Summary
-------
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
-----
/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/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/Util.java 1339793
Diff: https://reviews.apache.org/r/5159/diff
Testing
-------
test-commit + new test
Thanks,
Julien