> On May 17, 2013, 3:12 p.m., Cheolsoo Park wrote: > > src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/expressionOperators/POCast.java, > > lines 394-395 > > <https://reviews.apache.org/r/11203/diff/1/?file=293349#file293349line394> > > > > This isn't what you introduced, but I think this is incorrect. > > > > Shouldn't "in.getNextBigDecimal()" be "in.getNextBigInteger()" since > > we're casting BI to BD here?
Yep, looks like a bug. Good catch! > On May 17, 2013, 3:12 p.m., Cheolsoo Park wrote: > > src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/PhysicalOperator.java, > > lines 301-302 > > <https://reviews.apache.org/r/11203/diff/1/?file=293332#file293332line301> > > > > Can you fix indentation here? That because I generated the patch ignoring whitespace to make it more readable. I will commit with correct indentation - Julien ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11203/#review20674 ----------------------------------------------------------- On May 16, 2013, 9:35 p.m., Julien Le Dem wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/11203/ > ----------------------------------------------------------- > > (Updated May 16, 2013, 9:35 p.m.) > > > Review request for pig, Daniel Dai, Dmitriy Ryaboy, Cheolsoo Park, and Bill > Graham. > > > Description > ------- > > Refactor physical operators to remove methods parameters that are always null > > > This addresses bug PIG-3307. > https://issues.apache.org/jira/browse/PIG-3307 > > > Diffs > ----- > > > src/org/apache/pig/backend/hadoop/executionengine/mapReduceLayer/MergeJoinIndexer.java > d5aff3d > > src/org/apache/pig/backend/hadoop/executionengine/mapReduceLayer/PigCombiner.java > 6cfc8c0 > > src/org/apache/pig/backend/hadoop/executionengine/mapReduceLayer/PigGenericMapBase.java > 7c499f6 > > src/org/apache/pig/backend/hadoop/executionengine/mapReduceLayer/PigGenericMapReduce.java > 6145214 > > src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/PhysicalOperator.java > fc0112a > > src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/expressionOperators/Add.java > 5bceca6 > > src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/expressionOperators/BinaryComparisonOperator.java > 3e434f3 > > src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/expressionOperators/ComparisonOperator.java > 51d9f34 > > src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/expressionOperators/ConstantExpression.java > 7e4cffa > > src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/expressionOperators/Divide.java > bdcc72b > > src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/expressionOperators/EqualToExpr.java > a767c36 > > src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/expressionOperators/ExpressionOperator.java > 9cca2c3 > > src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/expressionOperators/GTOrEqualToExpr.java > b5e3c83 > > src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/expressionOperators/GreaterThanExpr.java > f3b5d44 > > src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/expressionOperators/LTOrEqualToExpr.java > 35786c0 > > src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/expressionOperators/LessThanExpr.java > c9b3157 > > src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/expressionOperators/Mod.java > 1108846 > > src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/expressionOperators/Multiply.java > 2795b78 > > src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/expressionOperators/NotEqualToExpr.java > 294f84a > > src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/expressionOperators/POAnd.java > f24c2ac > > src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/expressionOperators/POBinCond.java > 312f3ac > > src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/expressionOperators/POCast.java > 987cc21 > > src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/expressionOperators/POIsNull.java > 9ea89f7 > > src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/expressionOperators/POMapLookUp.java > fd5573f > > src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/expressionOperators/PONegative.java > 8d3fcb1 > > src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/expressionOperators/PONot.java > 973dfc5 > > src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/expressionOperators/POOr.java > 498eb12 > > src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/expressionOperators/POProject.java > 8886df7 > > src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/expressionOperators/PORegexp.java > 6634915 > > src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/expressionOperators/PORelationToExprProject.java > e400a95 > > src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/expressionOperators/POUserComparisonFunc.java > 1aa1671 > > src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/expressionOperators/POUserFunc.java > 167cf06 > > src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/expressionOperators/Subtract.java > 4999995 > > src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/relationalOperators/POCollectedGroup.java > a5adaf7 > > src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/relationalOperators/POCombinerPackage.java > 4a58a7e > > src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/relationalOperators/POCounter.java > 30dcea2 > > src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/relationalOperators/POCross.java > b90b0a2 > > src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/relationalOperators/PODemux.java > e26c611 > > src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/relationalOperators/PODistinct.java > ed2d39e > > src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/relationalOperators/POFRJoin.java > a4abdd8 > > src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/relationalOperators/POFilter.java > 552d6fa > > src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/relationalOperators/POForEach.java > 45de398 > > src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/relationalOperators/POGlobalRearrange.java > 5fac709 > > src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/relationalOperators/POJoinPackage.java > 7e357ec > > src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/relationalOperators/POLimit.java > 3a2542b > > src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/relationalOperators/POLoad.java > 55bd896 > > src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/relationalOperators/POLocalRearrange.java > 88621ba > > src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/relationalOperators/POMergeCogroup.java > eea840a > > src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/relationalOperators/POMergeJoin.java > b6a4d96 > > src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/relationalOperators/POMultiQueryPackage.java > 57a56fa > > src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/relationalOperators/POOptimizedForEach.java > 91b3f00 > > src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/relationalOperators/POPackage.java > e755e2a > > src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/relationalOperators/POPackageLite.java > e17d547 > > src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/relationalOperators/POPartialAgg.java > b909dc0 > > src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/relationalOperators/POPartitionRearrange.java > 5227d55 > > src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/relationalOperators/POPreCombinerLocalRearrange.java > 59b032f > > src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/relationalOperators/PORank.java > 9e3d1eb > > src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/relationalOperators/POSort.java > c76236b > > src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/relationalOperators/POSortedDistinct.java > 8000c9a > > src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/relationalOperators/POSplit.java > de2c972 > > src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/relationalOperators/POStore.java > 249aecb > > src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/relationalOperators/POStream.java > 894dc7d > > src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/relationalOperators/POUnion.java > 508daef > src/org/apache/pig/impl/builtin/DefaultIndexableLoader.java e2d3b86 > test/org/apache/pig/test/PORead.java e4a07dd > test/org/apache/pig/test/TestAdd.java a80ef81 > test/org/apache/pig/test/TestBoolean.java 4e70e21 > test/org/apache/pig/test/TestCharArrayToNumeric.java 473599d > test/org/apache/pig/test/TestConstExpr.java d18c419 > test/org/apache/pig/test/TestDivide.java 48b1591 > test/org/apache/pig/test/TestEqualTo.java 2e3584b > test/org/apache/pig/test/TestFRJoin.java 2bf868e > test/org/apache/pig/test/TestFilter.java 9e6ce32 > test/org/apache/pig/test/TestForEach.java 317b9e7 > test/org/apache/pig/test/TestGTOrEqual.java c1a3f6d > test/org/apache/pig/test/TestGreaterThan.java 77bd3f0 > test/org/apache/pig/test/TestLTOrEqual.java 84a72ee > test/org/apache/pig/test/TestLessThan.java d57cbb7 > test/org/apache/pig/test/TestLoad.java 6faf94e > test/org/apache/pig/test/TestLocalRearrange.java b471463 > test/org/apache/pig/test/TestMod.java 995a6ba > test/org/apache/pig/test/TestMultiply.java deeba8b > test/org/apache/pig/test/TestNotEqualTo.java 6417d68 > test/org/apache/pig/test/TestNull.java 17f4408 > test/org/apache/pig/test/TestPOBinCond.java 65c6a0d > test/org/apache/pig/test/TestPOCast.java 1d5f7e9 > test/org/apache/pig/test/TestPODistinct.java 3168486 > test/org/apache/pig/test/TestPOGenerate.java cfe2347 > test/org/apache/pig/test/TestPOMapLookUp.java 7c6cd64 > test/org/apache/pig/test/TestPONegative.java 2e06947 > test/org/apache/pig/test/TestPOPartialAgg.java 6645952 > test/org/apache/pig/test/TestPOSort.java 756865e > test/org/apache/pig/test/TestPOSplit.java 71fb089 > test/org/apache/pig/test/TestPOUserFunc.java 77eab4e > test/org/apache/pig/test/TestPackage.java 984cd49 > test/org/apache/pig/test/TestProject.java ff7eedf > test/org/apache/pig/test/TestRegexp.java 0737f5e > test/org/apache/pig/test/TestSubtract.java f0a253b > test/org/apache/pig/test/TestUnion.java 339b459 > test/org/apache/pig/test/utils/POCastDummy.java b83ed51 > test/org/apache/pig/test/utils/TestHelper.java 1efabbe > > Diff: https://reviews.apache.org/r/11203/diff/ > > > Testing > ------- > > test-commit > > > Thanks, > > Julien Le Dem > >