----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19094/#review66847 -----------------------------------------------------------
client/src/main/java/org/apache/falcon/cli/FalconCLI.java <https://reviews.apache.org/r/19094/#comment110510> Why "touch" instead of "forceupdate"? Wouldn't it be better to use the requested action rather than refer to the trigger event that could cause it? docs/src/site/twiki/FalconCLI.twiki <https://reviews.apache.org/r/19094/#comment110512> Maybe "Force Update operation updates an already submitted/scheduled entity, without changing it, causing any update-triggered side effects." docs/src/site/twiki/restapi/EntityTouch.twiki <https://reviews.apache.org/r/19094/#comment110511> Shouldn't there be a "</verbatim>" here? - Scott Preece On Jan. 6, 2015, 8:41 a.m., pavan kumar kolamuri wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/19094/ > ----------------------------------------------------------- > > (Updated Jan. 6, 2015, 8:41 a.m.) > > > Review request for Falcon and shwethags. > > > Repository: falcon-git > > > Description > ------- > > Add touch option to force update an entity in workflow engine(even without > any changes to entity) > > > Diffs > ----- > > client/src/main/java/org/apache/falcon/cli/FalconCLI.java ca514c1 > client/src/main/java/org/apache/falcon/client/FalconClient.java 7f1bc27 > > common/src/main/java/org/apache/falcon/workflow/engine/AbstractWorkflowEngine.java > f5b142b > docs/src/site/twiki/FalconCLI.twiki d37cf8c > docs/src/site/twiki/restapi/EntityTouch.twiki PRE-CREATION > > oozie/src/main/java/org/apache/falcon/workflow/engine/OozieWorkflowEngine.java > f1f96b1 > prism/src/main/java/org/apache/falcon/resource/AbstractEntityManager.java > 4a686e7 > > prism/src/main/java/org/apache/falcon/resource/AbstractSchedulableEntityManager.java > a4d1f8b > > prism/src/main/java/org/apache/falcon/resource/proxy/SchedulableEntityManagerProxy.java > cfa70a0 > > webapp/src/main/java/org/apache/falcon/resource/SchedulableEntityManager.java > 2ec7f66 > webapp/src/test/java/org/apache/falcon/cli/FalconCLIIT.java 118003f > webapp/src/test/java/org/apache/falcon/resource/EntityManagerJerseyIT.java > 40f8e04 > > Diff: https://reviews.apache.org/r/19094/diff/ > > > Testing > ------- > > Unit testing and end to end testing done > > > Thanks, > > pavan kumar kolamuri > >
