Re: [eclipselink-dev] Opinion request: Persisting non-entity subclass of

[Gordon Yorke <gordon.yorke@xxxxxxxxxx>]

This patch should be isolated to the JPA api's.  As implemented it will 
break Glassfish bug 1021 (unless users set the internal property) and 
cause issues if EclipseLink is used in the 2.1 EJB functionality.
--Gordon

Tom Ware wrote:
> Hi James,
>
>   Could you review the patch for proposed fix?
>
> https://bugs.eclipse.org/bugs/show_bug.cgi?id=237281
>
> -Tom
>
> JAMES.SUTHERLAND@xxxxxxxxxx wrote:
> > I can be useful in cases.  Perhaps at a minimum keep an 
> > InheritancePolicy option to allow the existing functionality.
> >
> > I think we used to rely on this functionality for CMP generated 
> > subclasses.
> >
> >
> >
> > -----Original Message-----
> > From: TOM.WARE@xxxxxxxxxx Sent: Tuesday, June 17, 2008 1:17 PM
> > To: Dev mailing list for Eclipse Persistence Services
> > Subject: [eclipselink-dev] Opinion request: Persisting non-entity 
> > subclass of Entity
> >
> > Hi Folks,
> >
> >    I am wondering if anyone has an opinion about the following:
> >
> >    At the moment, EclipseLink does not throw an exception when you 
> > persist a
> > subclass of an Entity that is not annotated as an Entity.  Instead, 
> > it treats it
> > as though it is the superclass and persists the superclass.
> >
> >    The following bug has been entered because of this issue:
> >
> > https://bugs.eclipse.org/bugs/show_bug.cgi?id=237281
> >
> >    Note: Although the bug claims we throw the incorrect exception in 
> > this case,
> > my recreation of this issue shows we continue without any issue at all.
> >
> >    It seems that although the EclipseLink behavior actually allows some
> > additional use cases not supported by the spec, this behavior is, 
> > strictly
> > speaking, a spec violation.
> >
> >    Does anyone have an opinion about whether we should keep the 
> > current behavior
> > as an option?
> >
> > Thanks in advance,
> > Tom
> > _______________________________________________
> > eclipselink-dev mailing list
> > eclipselink-dev@xxxxxxxxxxx
> > https://dev.eclipse.org/mailman/listinfo/eclipselink-dev
> > _______________________________________________
> > eclipselink-dev mailing list
> > eclipselink-dev@xxxxxxxxxxx
> > https://dev.eclipse.org/mailman/listinfo/eclipselink-dev
> _______________________________________________
> eclipselink-dev mailing list
> eclipselink-dev@xxxxxxxxxxx
> https://dev.eclipse.org/mailman/listinfo/eclipselink-dev