> On Dec. 3, 2014, 12:49 p.m., Ruslan Ostafiychuk wrote: > > falcon-regression/merlin/src/test/java/org/apache/falcon/regression/CombinedActionsTest.java, > > line 214 > > <https://reviews.apache.org/r/25336/diff/5/?file=781536#file781536line214> > > > > we are trying to aviod using runRemoteScriptAsSudo() > > this code will not work on Windows > > > > Looks like this can be done manually with single 'ssh-copy-id' command
Didn't want the test to expect anything that has to be done manually. Can you suggest something for windows? - Karishma ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25336/#review63666 ----------------------------------------------------------- On Dec. 30, 2014, 9:44 a.m., Karishma Gulati wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/25336/ > ----------------------------------------------------------- > > (Updated Dec. 30, 2014, 9:44 a.m.) > > > Review request for Falcon. > > > Repository: falcon-git > > > Description > ------- > > A test needs to be added where one worflow has various possible actions like > mr, hive, pig. So far, we just specify the engine type as hive or pig for > such specific tests. > > > Diffs > ----- > > > falcon-regression/merlin-core/src/main/java/org/apache/falcon/regression/core/util/BundleUtil.java > 6e25a60 > > falcon-regression/merlin-core/src/main/java/org/apache/falcon/regression/core/util/ExecUtil.java > 82c8919 > > falcon-regression/merlin-core/src/main/java/org/apache/falcon/regression/core/util/HCatUtil.java > d878ecb > > falcon-regression/merlin-core/src/main/java/org/apache/falcon/regression/core/util/OSUtil.java > ed29d07 > > falcon-regression/merlin/src/test/java/org/apache/falcon/regression/CombinedActionsTest.java > PRE-CREATION > falcon-regression/merlin/src/test/resources/combinedActions/cluster-0.1.xml > PRE-CREATION > > falcon-regression/merlin/src/test/resources/combinedActions/feed-template1.xml > PRE-CREATION > > falcon-regression/merlin/src/test/resources/combinedActions/feed-template1_hcat.xml > PRE-CREATION > > falcon-regression/merlin/src/test/resources/combinedActions/feed-template2.xml > PRE-CREATION > > falcon-regression/merlin/src/test/resources/combinedActions/feed-template2_hcat.xml > PRE-CREATION > > falcon-regression/merlin/src/test/resources/combinedActions/feed-template3.xml > PRE-CREATION > falcon-regression/merlin/src/test/resources/combinedActions/process-agg.xml > PRE-CREATION > falcon-regression/merlin/src/test/resources/combinedWorkflow/id.pig > PRE-CREATION > falcon-regression/merlin/src/test/resources/combinedWorkflow/script.hql > PRE-CREATION > falcon-regression/merlin/src/test/resources/combinedWorkflow/workflow.xml > PRE-CREATION > > Diff: https://reviews.apache.org/r/25336/diff/ > > > Testing > ------- > > > Thanks, > > Karishma Gulati > >
