dev@glassfish.java.net

Re: Issue 257

From: Amy Roh <Amelia.Roh_at_Sun.COM>
Date: Wed, 15 Feb 2006 10:09:39 -0800

Hi Nathaniel,

Nathaniel Stoddard wrote:

>This is related to issue 257 (entity-persistence related). (Sorry if
>this is going to the wrong place.) I was going to add a comment to
>the existing issue, but couldn't figure out how or if it was possible.
>
>
You're probably not seeing the "submit" button with additional comments,
etc. because you're not logged in. Once you log in with your java.net
account, the issue page should be very clear and let you submit
comments, etc. Please try this and let me know. It's great you're
discussing this but it'll be good to save your comments with the issue
so others can follow up on it.

Thanks,
Amy

> The issue says that a NullPointerException is being caused by
>abstract classes that extend concrete classes. The abstract class
>doesn't provide a @DiscriminatorValue and the code throws the
>exception when it expects one.
>
>I believe the issue goes farther than that. The Javadoc for the
>function throwing the error says that a DiscriminatorValue will
>default to the class name if the annotation isn't present. But it
>doesn't, ergo the NullPointerException.
>
>I think this is really two issues:
>
>1. The code should default the DiscriminatorValue to the class name
>as it says it will.
>2. Abstract classes shouldn't require (or default to class name) the
>DiscriminatorValue at all. This may not even be an issue if #1 is
>fixed.
>
>What do you folks think?
>
>---------------------------------------------------------------------
>To unsubscribe, e-mail: dev-unsubscribe_at_glassfish.dev.java.net
>For additional commands, e-mail: dev-help_at_glassfish.dev.java.net
>
>
>