----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38294/#review99191 -----------------------------------------------------------
common/src/main/java/org/apache/falcon/entity/EntityUtil.java (line 908) <https://reviews.apache.org/r/38294/#comment156103> Minor nit: update the message in exception as keyvalue[0] or keyvalue[1] can be invalid webapp/src/test/java/org/apache/falcon/resource/TestContext.java (line 295) <https://reviews.apache.org/r/38294/#comment156109> Minor nit: Can you use StringUtils.isNotEmpty instead? - Sowmya Ramesh On Sept. 14, 2015, 6:21 a.m., Pallavi Rao wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/38294/ > ----------------------------------------------------------- > > (Updated Sept. 14, 2015, 6:21 a.m.) > > > Review request for Falcon. > > > Bugs: FALCON-1434 > https://issues.apache.org/jira/browse/FALCON-1434 > > > Repository: falcon-git > > > Description > ------- > > The schedule API will be enhanced to accept a key-value properties. This is a > foundation to enable users to specify the scheduler on which they want to > schedule the entity. This in turn enables migration to native scheduler from > Oozie. > Example: > bin/falcon entity -schedule -props falcon.scheduler=native -name <entity name> > > > Diffs > ----- > > client/src/main/java/org/apache/falcon/cli/FalconCLI.java d4da302 > client/src/main/java/org/apache/falcon/client/AbstractFalconClient.java > 282b41b > client/src/main/java/org/apache/falcon/client/FalconClient.java 44436d2 > common/src/main/java/org/apache/falcon/entity/EntityUtil.java 25d9008 > > common/src/main/java/org/apache/falcon/workflow/engine/AbstractWorkflowEngine.java > ea86c2a > common/src/test/java/org/apache/falcon/entity/EntityUtilTest.java cfdc84d > > oozie/src/main/java/org/apache/falcon/workflow/engine/OozieWorkflowEngine.java > 5f79ca1 > > prism/src/main/java/org/apache/falcon/resource/AbstractSchedulableEntityManager.java > f9405dc > > prism/src/main/java/org/apache/falcon/resource/proxy/SchedulableEntityManagerProxy.java > ceabb06 > unit/src/main/java/org/apache/falcon/unit/FalconUnitClient.java eb65cb3 > unit/src/test/java/org/apache/falcon/unit/FalconUnitTestBase.java 997b301 > unit/src/test/java/org/apache/falcon/unit/TestFalconUnit.java 498f50e > > webapp/src/main/java/org/apache/falcon/resource/SchedulableEntityManager.java > 1f8cc1b > webapp/src/test/java/org/apache/falcon/cli/FalconCLIIT.java 0062070 > webapp/src/test/java/org/apache/falcon/resource/EntityManagerJerseyIT.java > bcd3bd5 > webapp/src/test/java/org/apache/falcon/resource/TestContext.java 54671fb > > Diff: https://reviews.apache.org/r/38294/diff/ > > > Testing > ------- > > UT added > Manually tested to ensure CLI accepts properties and it is propagated. > > > Thanks, > > Pallavi Rao > >
