> On Oct. 3, 2012, 6:29 p.m., Julien Le Dem wrote: > > src/org/apache/pig/data/SchemaTupleFrontend.java, lines 124-125 > > <https://reviews.apache.org/r/7336/diff/1/?file=160423#file160423line124> > > > > why did you add this? > > Jonathan Coveney wrote: > Whoops. This is an artifact of some bugtesting.
It turns out that it is in fact necessary. There are some code paths where it still needs to read this flag, but without short circuiting viz. local mode. - Jonathan ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/7336/#review12139 ----------------------------------------------------------- On Sept. 28, 2012, 12:21 a.m., Jonathan Coveney wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/7336/ > ----------------------------------------------------------- > > (Updated Sept. 28, 2012, 12:21 a.m.) > > > Review request for pig. > > > Description > ------- > > This uses the SchemaTuple in Foreach, which means it will work after Loads > > > This addresses bug PIG-2877. > https://issues.apache.org/jira/browse/PIG-2877 > > > Diffs > ----- > > src/org/apache/pig/PigConfiguration.java bf13d9d > src/org/apache/pig/PigConstants.java PRE-CREATION > src/org/apache/pig/backend/hadoop/executionengine/HExecutionEngine.java > d5c9315 > src/org/apache/pig/backend/hadoop/executionengine/HJob.java 2ff294a > > src/org/apache/pig/backend/hadoop/executionengine/mapReduceLayer/JobControlCompiler.java > 86d04f8 > > src/org/apache/pig/backend/hadoop/executionengine/mapReduceLayer/PigGenericMapBase.java > 6ab33a4 > > src/org/apache/pig/backend/hadoop/executionengine/mapReduceLayer/PigGenericMapReduce.java > 2e70462 > > src/org/apache/pig/backend/hadoop/executionengine/mapReduceLayer/partitioners/WeightedRangePartitioner.java > d7504ae > > src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/PhysicalOperator.java > 26d5389 > > src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/relationalOperators/POForEach.java > 4e6876a > > src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/relationalOperators/POSort.java > 702a01b > src/org/apache/pig/data/BinInterSedes.java a7ba39b > src/org/apache/pig/data/SchemaTuple.java 4f04834 > src/org/apache/pig/data/SchemaTupleBackend.java 2746316 > src/org/apache/pig/data/SchemaTupleClassGenerator.java ef15318 > src/org/apache/pig/data/SchemaTupleFactory.java b467d55 > src/org/apache/pig/data/SchemaTupleFrontend.java a8bfbab > src/org/apache/pig/impl/builtin/FindQuantiles.java 1fcbdb1 > src/org/apache/pig/impl/io/ReadToEndLoader.java afb6ebd > > src/org/apache/pig/newplan/logical/relational/LogToPhyTranslationVisitor.java > 5c3dc43 > src/org/apache/pig/parser/LogicalPlanBuilder.java b0a8aa0 > test/org/apache/pig/data/TestSchemaTuple.java 24142c2 > test/org/apache/pig/test/TestBuiltin.java a835993 > test/org/apache/pig/test/TestCommit.java 5baaedc > test/org/apache/pig/test/TestExampleGenerator.java 4babccb > test/org/apache/pig/test/TestLoad.java a34e2bb > test/org/apache/pig/test/TestPigServer.java 18a3dff > test/org/apache/pig/test/TestPigServerWithMacros.java c6d8c4e > > Diff: https://reviews.apache.org/r/7336/diff/ > > > Testing > ------- > > > Thanks, > > Jonathan Coveney > >
