Hi Craig,

sorry for not replying earier.

Javadogs,

I've made these changes to the final spec.

Craig

On Feb 15, 2006, at 11:37 AM, Craig L Russell wrote:

1. <!ATTLIST column allows-null CDATA #IMPLIED>
should be
<!ATTLIST column allows-null (true|false) #IMPLIED>

2. <!ELEMENT foreign-key (extension*, (column|field|property)*, extension*)>
should be
<!ELEMENT foreign-key (extension*, (column*|field*|property*), extension*)>

I read (column*|field*|property*) as: either a list of columns or a list of fields or a list of properties. So you cannot have fields and properties mixed. Did you intend to have a list of columns, followed by a list of fields, followed by a list of properties: <!ELEMENT foreign-key (extension*, column*, field*, property*, extension*)>

The fetch-group element looks similar. Did you paln you change this too:
<!ELEMENT fetch-group (extension*, (fetch-group|field|property)*, extension*)>

Regards Michael


3. same as 2 for index


Same as 2 for unique as well.


4. <!ATTLIST element update-action CDATA #IMPLIED>
element update-action should restrict the permitted values to the same values as foreign-key update-action

5. Missing element key attribute update-action

6. Missing element value attribute update-action

7. Element order should not have the delete-action, update-action, indexed, and unique attributes
Anything else?

Craig

Craig Russell
Architect, Sun Java Enterprise System http://java.sun.com/products/jdo
408 276-5638 mailto:[EMAIL PROTECTED]
P.S. A good JDO? O, Gasp!


Craig Russell

Architect, Sun Java Enterprise System http://java.sun.com/products/jdo

408 276-5638 mailto:[EMAIL PROTECTED]

P.S. A good JDO? O, Gasp!




--
Michael Bouschen                [EMAIL PROTECTED] Engineering GmbH
mailto:[EMAIL PROTECTED]        http://www.tech.spree.de/
Tel.:++49/30/235 520-33         Buelowstr. 66                   
Fax.:++49/30/2175 2012          D-10783 Berlin                  

Reply via email to