----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/4835/#review7266 -----------------------------------------------------------
Ship it! +1 my bad, patch works jus fine, thxs - Alejandro On 2012-04-25 02:47:23, Han Xiao wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/4835/ > ----------------------------------------------------------- > > (Updated 2012-04-25 02:47:23) > > > Review request for oozie. > > > Summary > ------- > > Schema of fs action require the fs operations must follow the sequence of > "delete, mkdir, move, chmod". > The limitation is not reasonable, oozie should allow fs operations to be done > in any sequence users like to. > > > This addresses bug OOZIE-817. > https://issues.apache.org/jira/browse/OOZIE-817 > > > Diffs > ----- > > > http://svn.apache.org/repos/asf/incubator/oozie/trunk/core/src/test/resources/wf-schema-valid.xml > 1327027 > > http://svn.apache.org/repos/asf/incubator/oozie/trunk/core/src/test/java/org/apache/oozie/service/TestLiteWorkflowAppService.java > 1327027 > > http://svn.apache.org/repos/asf/incubator/oozie/trunk/client/src/main/resources/oozie-workflow-0.3.xsd > 1327027 > > http://svn.apache.org/repos/asf/incubator/oozie/trunk/client/src/main/resources/oozie-workflow-0.2.xsd > 1327027 > > http://svn.apache.org/repos/asf/incubator/oozie/trunk/client/src/main/resources/oozie-workflow-0.2.5.xsd > 1327027 > > http://svn.apache.org/repos/asf/incubator/oozie/trunk/client/src/main/resources/oozie-workflow-0.1.xsd > 1327027 > > Diff: https://reviews.apache.org/r/4835/diff > > > Testing > ------- > > * Revise the TestLiteWorkflowAppService to add testing for fs action's schema. > * All testcase are passed. > > > Thanks, > > Han > >
