dev@glassfish.java.net

Re: Issue 257

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

You must not be an observer for the project, right? You can request to
be an observer and you'll have permission to add comments to issue tracker.

Amy

Nathaniel Stoddard wrote:

>I'm definitely logged in, but there's no "comment" or "submit" button
>anywhere! I noticed on the "permissions" page, I don't seem to have
>any permission indicating I can add a comment to an existing issue.
>
>On 2/15/06, Amy Roh <Amelia.Roh_at_sun.com> wrote:
>
>
>>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
>>>
>>>
>>>
>>>
>>>
>>
>>