----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38294/#review98645 -----------------------------------------------------------
Couple of queries 1> key-value properties - Is it not required to allow this for already scheduled entities? i.e. should be allowed for update operation too 2> Does any property passed in key-value properties requires validation? Was wondering if it is required for validate. common/src/main/java/org/apache/falcon/entity/EntityUtil.java (line 905) <https://reviews.apache.org/r/38294/#comment155155> Can keyValue[0] be empty? common/src/test/java/org/apache/falcon/entity/EntityUtilTest.java (line 338) <https://reviews.apache.org/r/38294/#comment155157> You can move invalid prop test to new test and use @Test expectedExceptions and expectedExceptionsMessageRegExp - much cleaner way. - Sowmya Ramesh On Sept. 11, 2015, 7:28 a.m., Pallavi Rao wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/38294/ > ----------------------------------------------------------- > > (Updated Sept. 11, 2015, 7:28 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 > > 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 > >
