-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/7771/
-----------------------------------------------------------

Review request for pig, Julien Le Dem, Cheolsoo Park, and Gianmarco De 
Francisci Morales.


Description
-------

I refactored the remaining tests. I did not remove MiniCluster usage.


This addresses bug PIG-3016.
    https://issues.apache.org/jira/browse/PIG-3016


Diffs
-----

  test/org/apache/pig/test/PigExecTestCase.java 32a502c 
  test/org/apache/pig/test/TestAccumulator.java 73ba0ce 
  test/org/apache/pig/test/TestAdd.java cdeabb3 
  test/org/apache/pig/test/TestAlgebraicEval.java 0bbd83d 
  test/org/apache/pig/test/TestAlgebraicEvalLocal.java df4b76a 
  test/org/apache/pig/test/TestBagFormat.java 09298d4 
  test/org/apache/pig/test/TestBatchAliases.java 6e952c7 
  test/org/apache/pig/test/TestBestFitCast.java a1bf5c5 
  test/org/apache/pig/test/TestBloom.java e9fe355 
  test/org/apache/pig/test/TestBoolean.java b3f9e3e 
  test/org/apache/pig/test/TestCharArrayToNumeric.java bd0880d 
  test/org/apache/pig/test/TestCmdLineParser.java eb91cae 
  test/org/apache/pig/test/TestCombiner.java af7b678 
  test/org/apache/pig/test/TestCompressedFiles.java d54ffaa 
  test/org/apache/pig/test/TestConstExpr.java a7a0e3b 
  test/org/apache/pig/test/TestConversions.java 152ad5c 
  test/org/apache/pig/test/TestDataModel.java d04658c 
  test/org/apache/pig/test/TestDeleteOnFail.java 7070285 
  test/org/apache/pig/test/TestDivide.java be7c6a3 
  test/org/apache/pig/test/TestEqualTo.java edefa35 
  test/org/apache/pig/test/TestFRJoin.java 829b8ae 
  test/org/apache/pig/test/TestFilter.java e954ea3 
  test/org/apache/pig/test/TestFilterOpNumeric.java 730e808 
  test/org/apache/pig/test/TestFilterOpString.java b65965f 
  test/org/apache/pig/test/TestFilterSimplification.java ade97b6 
  test/org/apache/pig/test/TestFilterUDF.java ffc589d 
  test/org/apache/pig/test/TestFinish.java d213a1d 
  test/org/apache/pig/test/TestForEach.java e47646f 
  test/org/apache/pig/test/TestForEachNestedPlanLocal.java a78568e 
  test/org/apache/pig/test/TestFuncSpec.java bc7144c 
  test/org/apache/pig/test/TestGTOrEqual.java a35bc74 
  test/org/apache/pig/test/TestGreaterThan.java d522975 
  test/org/apache/pig/test/TestInfixArithmetic.java cdf6948 
  test/org/apache/pig/test/TestInputOutputFileValidator.java 67b2873 
  test/org/apache/pig/test/TestInputOutputMiniClusterFileValidator.java caa62cb 
  test/org/apache/pig/test/TestInstantiateFunc.java 31c37b1 
  test/org/apache/pig/test/TestJoin.java a4f3aff 
  test/org/apache/pig/test/TestJoinSmoke.java afe7e60 
  test/org/apache/pig/test/TestKeyTypeDiscoveryVisitor.java 2bbeca1 
  test/org/apache/pig/test/TestLOLoadDeterminedSchema.java 2177b58 
  test/org/apache/pig/test/TestLTOrEqual.java 6b7c795 
  test/org/apache/pig/test/TestLargeFile.java 79590ce 
  test/org/apache/pig/test/TestLessThan.java 7df1dc1 
  test/org/apache/pig/test/TestLocal.java 5680196 
  test/org/apache/pig/test/TestLocal2.java eea7b2f 
  test/org/apache/pig/test/TestMapReduce2.java 30574db 
  test/org/apache/pig/test/TestMod.java 523238c 
  test/org/apache/pig/test/TestMultiply.java 7cb1d41 
  test/org/apache/pig/test/TestNewPlanColumnPrune.java bed006e 
  test/org/apache/pig/test/TestNewPlanListener.java 7701182 
  test/org/apache/pig/test/TestNewPlanLogicalOptimizer.java 5eb841d 
  test/org/apache/pig/test/TestNewPlanOperatorPlan.java 1f8fe56 
  test/org/apache/pig/test/TestNewPlanPruneMapKeys.java d1cce22 
  test/org/apache/pig/test/TestNewPlanRule.java 4a7ff0a 
  test/org/apache/pig/test/TestNotEqualTo.java e6c3bca 
  test/org/apache/pig/test/TestNull.java 64deaaf 
  test/org/apache/pig/test/TestNullConstant.java 3ae25d9 
  test/org/apache/pig/test/TestOrderBy2.java 4ee4f26 
  test/org/apache/pig/test/TestOrderBy3.java 2067d7a 
  test/org/apache/pig/test/TestPOBinCond.java 20bd734 
  test/org/apache/pig/test/TestPODistinct.java 60f9d73 
  test/org/apache/pig/test/TestPOGenerate.java e0fd796 
  test/org/apache/pig/test/TestPOMapLookUp.java 3ed0900 
  test/org/apache/pig/test/TestPONegative.java 220c409 
  test/org/apache/pig/test/TestPORegexp.java d6e15ac 
  test/org/apache/pig/test/TestPOSort.java 600ee0c 
  test/org/apache/pig/test/TestPOUserFunc.java 3a90d6c 
  test/org/apache/pig/test/TestPackage.java 7bc6c5c 
  test/org/apache/pig/test/TestParamSubPreproc.java 1a52691 
  test/org/apache/pig/test/TestParser.java 17dc42a 
  test/org/apache/pig/test/TestPhyOp.java 0c9ecb0 
  test/org/apache/pig/test/TestPi.java f0883d1 
  test/org/apache/pig/test/TestPigContext.java ca0ef84 
  test/org/apache/pig/test/TestPigProgressReporting.java e4f76ec 
  test/org/apache/pig/test/TestPigScriptParser.java 2acb1a8 
  test/org/apache/pig/test/TestPigSplit.java af70e9d 
  test/org/apache/pig/test/TestPinOptions.java a730ce7 
  test/org/apache/pig/test/TestPlanGeneration.java c1bc806 
  test/org/apache/pig/test/TestPoissonSampleLoader.java 93cf24f 
  test/org/apache/pig/test/TestProject.java 0b4194d 
  test/org/apache/pig/test/TestPruneColumn.java 03139a5 
  test/org/apache/pig/test/TestRank1.java fbc6a7d 
  test/org/apache/pig/test/TestRank2.java d4daf8b 
  test/org/apache/pig/test/TestRank3.java 6dd2624 
  test/org/apache/pig/test/TestRegexp.java d5e1327 
  test/org/apache/pig/test/TestRelationToExprProject.java 1411451 
  test/org/apache/pig/test/TestSchemaUtil.java e1d1133 
  test/org/apache/pig/test/TestSecondarySort.java 1661b8d 
  test/org/apache/pig/test/TestSkewedJoin.java eaf08d1 
  test/org/apache/pig/test/TestSplitStore.java 1a94b2b 
  test/org/apache/pig/test/TestStore.java 7f1c77b 
  test/org/apache/pig/test/TestStoreOld.java 37ad3bf 
  test/org/apache/pig/test/TestStreamingLocal.java b745074 
  test/org/apache/pig/test/TestSubtract.java 25900d0 
  test/org/apache/pig/test/TestTextDataParser.java db221ff 
  test/org/apache/pig/test/TestToolsPigServer.java e021b8c 
  test/org/apache/pig/test/TestTypeCheckingValidatorNewLP.java 356e711 
  test/org/apache/pig/test/TestUDF.java f1b10f8 
  test/org/apache/pig/test/TestUDFGroovy.java e5b8c8e 
  test/org/apache/pig/test/TestUDFWithoutParameter.java 2527afa 
  test/org/apache/pig/test/TestUTF8.java 42aab25 
  test/org/apache/pig/test/TestUnion.java 8da2bb7 
  test/org/apache/pig/test/pigunit/TestPigTest.java 65cd163 

Diff: https://reviews.apache.org/r/7771/diff/


Testing
-------

All tests pass on my local machine.


Thanks,

Jonathan Coveney

Reply via email to