[ 
https://issues.apache.org/jira/browse/OOZIE-817?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13260176#comment-13260176
 ] 

[email protected] commented on OOZIE-817:
-----------------------------------------------------



bq.  On 2012-04-23 17:46:35, Mona Chitnis wrote:
bq.  > 
http://svn.apache.org/repos/asf/incubator/oozie/trunk/core/src/test/resources/wf-schema-valid.xml,
 line 94
bq.  > <https://reviews.apache.org/r/4835/diff/1/?file=103688#file103688line94>
bq.  >
bq.  >     whitespace

have fixed it


bq.  On 2012-04-23 17:46:35, Mona Chitnis wrote:
bq.  > 
http://svn.apache.org/repos/asf/incubator/oozie/trunk/core/src/test/java/org/apache/oozie/service/TestLiteWorkflowAppService.java,
 line 288
bq.  > <https://reviews.apache.org/r/4835/diff/1/?file=103687#file103687line288>
bq.  >
bq.  >     remove extra whitespaces.

have fixed it


bq.  On 2012-04-23 17:46:35, Mona Chitnis wrote:
bq.  > 
http://svn.apache.org/repos/asf/incubator/oozie/trunk/client/src/main/resources/oozie-workflow-0.1.xsd,
 line 187
bq.  > <https://reviews.apache.org/r/4835/diff/1/?file=103683#file103683line187>
bq.  >
bq.  >     Is occurrence of each operation limited to once?

I am afraid there is no need. A choice element means that one and only one of 
elements in it must appear.
If there is another element, it should be treated as another choice element. 


- Han


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/4835/#review7145
-----------------------------------------------------------


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


                
> Fix the unreasonable definition of the schema to fs action
> ----------------------------------------------------------
>
>                 Key: OOZIE-817
>                 URL: https://issues.apache.org/jira/browse/OOZIE-817
>             Project: Oozie
>          Issue Type: Bug
>          Components: client
>    Affects Versions: trunk
>            Reporter: Han Xiao
>             Fix For: trunk
>
>
> 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 message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to