dev@glassfish.java.net

Re: Issue 257

From: Nathaniel Stoddard <nathaniel.stoddard_at_gmail.com>
Date: Wed, 15 Feb 2006 13:43:58 -0500

Ah, ok. How do I request to be an observer? Or is this email good
enough for that?

On 2/15/06, Amy Roh <Amelia.Roh_at_sun.com> wrote:
> 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
> >>>
> >>>
> >>>
> >>>
> >>>
> >>
> >>
>
>