Bug 4149

Summary: XSD inconsistencies
Product: jbatch Reporter: mminella
Component: sourceAssignee: cvignola
Status: CLOSED FIXED    
Severity: critical CC: issues, ScottKurz
Priority: P5    
Version: 1   
Target Milestone: ---   
Hardware: Macintosh   
OS: All   
Whiteboard:
Attachments: Working copy of RI XSD

Description mminella 2012-09-26 15:07:00 UTC
With the agreement that the XSD will be included as part of the specification, there are a number of inconsistencies between it and the spec as it is currently defined.  The list of issues below (while I'm sure not exhaustive) was compiled against the XSD provided on 9/25/2012:

- Property is incorrect
    - target missing
    - applyTo should not be there
    - applyToPartition should not be there
- Properties is incorrect
    - Missing partition
- Error handling is missing
    - Skipping exceptions (4.2.1.2.1)
    - Retryable exceptions (4.2.1.2.2)
    - Rollback exceptions (4.2.1.3)
- Partition is incorrect
    - threads does not belong.
- Flow is incorrect
    - does not allow a nested partition element (although, is this even needed?)
- Partition Mapper is missing
- Partition Reducer is missing
- Logical Transaction verbage still exists.
Comment 1 ScottKurz 2012-09-27 14:14:59 UTC
Michael, thanks for making those comments as those changes all were needed to the RI working copy of the XSD I shared.

One exception.. I didn't follow your comment about the flow not being able to support the nested partition... The flow already can contain a child step, and the partitioned aspect is configured with a <partition> child of <step>, right?

That said, the spec does in section 5.2.6 suggest that <partition> follow <chunk>/<batchlet>, unlike the existing RI XSD, so that's still something to clear up.

I have a separate bug (question) open (4148) about the no-rollback vs. retryable... so we haven't updated that part yet, but I'll attach the merge of the changes we made so far.

Scott
Comment 2 ScottKurz 2012-09-27 14:19:18 UTC
Created attachment 63 [details]
Working copy of RI XSD
Comment 3 cvignola 2013-01-16 15:19:37 UTC
Corrected XSD has been updatedin proposed final draft.