[ 
https://issues.apache.org/jira/browse/JDO-403?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12513769
 ] 

Craig Russell commented on JDO-403:
-----------------------------------

I notice that @ForeignKey, @Index, and @Unique use String [ ] fields() whereas 
@Embedded uses Field[ ] fields() as an element.

Should these be different, now that we have added String name() to @Field? 
Seems they should be consistent.

@Index(fields={
    "field1",
    "field2",
    "field3"})

versus

@Index(fields={
    @Field("field1"),
    @Field("field2"),
    @Field("field3")})

I guess it doesn't matter too much. There isn't any extra information needed 
from the field for Index, Unique, and ForeignKey...

> JDO2 Annotations
> ----------------
>
>                 Key: JDO-403
>                 URL: https://issues.apache.org/jira/browse/JDO-403
>             Project: JDO
>          Issue Type: New Feature
>          Components: api2
>    Affects Versions: JDO 2 final
>            Reporter: Andy Jefferson
>            Assignee: Michelle Caisse
>             Fix For: JDO 2 maintenance release 1
>
>         Attachments: annotations.patch, api2_property_table.patch, 
> embedded.patch, jdo_2_1_annotations.jar, tck2a.patch, tck2b.patch
>
>
> It would be desirable for JDO2 to have its own set of annotations. We have 
> developed a set within JPOX that would likely serve as a starting point for 
> such a set. In my opinion they should be
> 1. Split into javax.jdo.annotations.jdo and javax.jdo.annotations.orm
> 2. Move ORM attributes from some of the JDO annotations and have a ORM 
> annotation.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to