Negative VOTE Issue 141: Persistent property mapping

[Craig L Russell <Cr...@Sun.COM>]

Javadogs,

If you have any issues with the proposal please reply.

This proposal adds property to the metadata where only field  
previously was mentioned.

Craig

Issue  141
H
Add support for persistent properties
Erik Bengtson noticed that the metadata doesn't include enough  
metadata to map persistent properties
Index: src/dtd/javax/jdo/jdo.dtd
===================================================================
--- src/dtd/javax/jdo/jdo.dtd	(revision 332650)
+++ src/dtd/javax/jdo/jdo.dtd	(working copy)
@@ -18,11 +18,13 @@
  <!ATTLIST interface objectid-class CDATA #IMPLIED>
  <!ATTLIST interface requires-extent (true|false) 'true'>
  <!ATTLIST interface detachable (true|false) 'false'>
+<!ATTLIST interface embedded-only (true|false) #IMPLIED>
  <!ATTLIST interface catalog CDATA #IMPLIED>
  <!ATTLIST interface schema CDATA #IMPLIED>

  <!ELEMENT property (extension*, (array|collection|map)?, join?,  
embedded?, element?, key?, value?, order?, column*, foreign-key?,  
index?, unique?, extension*)>
  <!ATTLIST property name CDATA #REQUIRED>
+<!ATTLIST property persistence-modifier (persistent|transactional| 
none) #IMPLIED>
  <!ATTLIST property default-fetch-group (true|false) #IMPLIED>
  <!ATTLIST property load-fetch-group CDATA #IMPLIED>
  <!ATTLIST property null-value (default|exception|none) 'none'>
@@ -180,11 +182,11 @@
  <!ATTLIST order indexed (true|false|unique) #IMPLIED>
  <!ATTLIST order mapped-by CDATA #IMPLIED>

-<!ELEMENT fetch-group (fetch-group|field)*>
+<!ELEMENT fetch-group (extension*, (fetch-group|field|property)*,  
extension*)>
  <!ATTLIST fetch-group name CDATA #REQUIRED>
  <!ATTLIST fetch-group post-load (true|false) #IMPLIED>

-<!ELEMENT embedded (extension*, field*, extension*)>
+<!ELEMENT embedded (extension*, (field|property)*, extension*)>
  <!ATTLIST embedded owner-field CDATA #IMPLIED>
  <!ATTLIST embedded null-indicator-column CDATA #IMPLIED>
  <!ATTLIST embedded null-indicator-value CDATA #IMPLIED>
Index: src/dtd/javax/jdo/orm.dtd
===================================================================
--- src/dtd/javax/jdo/orm.dtd	(revision 331619)
+++ src/dtd/javax/jdo/orm.dtd	(working copy)
@@ -134,7 +134,7 @@
  <!ATTLIST order indexed (true|false|unique) #IMPLIED>
  <!ATTLIST order mapped-by CDATA #IMPLIED>

-<!ELEMENT embedded (extension*, field*, extension*)>
+<!ELEMENT embedded (extension*, (field|property)*, extension*)>
  <!ATTLIST embedded null-indicator-column CDATA #IMPLIED>
  <!ATTLIST embedded null-indicator-value CDATA #IMPLIED>
  <!ATTLIST embedded owner-field CDATA #IMPLIED>

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