RE: A couple of OSUser questions

"Brian Topping" <[email protected]> Sat, 6 Dec 2003 14:51:59 -0500
Newsgroups gmane.comp.java.open-symphony.devel
Message-ID <[email protected]>
Hi Eric,

The bug was USER-49. 

If CVS can generate a good patch, I'm all for it, but I've never mastered CVS
client that much.  If you can cook me a command, I'll give it a shot.  There
are a few file moves and renames in there.

At this point, my deltas have all the conversions for Object PKs.  So those
are going to cloud the deltas that I can generate.

thanks,

-b

> -----Original Message-----
> From: [email protected]
> [mailto:[email protected]]On
>  Behalf Of
> Eric Pugh
> Sent: Saturday, December 06, 2003 2:45 PM
> To: [email protected]
> Subject: RE: [Opensymphony-developers] A couple of OSUser questions
> 
> 
> Brian,
> 
> I actually lean towards the PK being an object as well.  I 
> prefer to be able to know that getting a PK which is null 
> means it hasn't been saved, versus knowing the 0 means 
> unsaved.  Especially as 0 is a perfectly good ID value!  
> 
> Which bug report was yours?  I'll look at your patch and 
> apply it, do you just want to one patch for all your changes?
> 
> Eric
> 
> > -----Original Message-----
> > From: [email protected]
> > [mailto:[email protected]]On
> >  Behalf Of
> > Brian Topping
> > Sent: Saturday, December 06, 2003 7:16 PM
> > To: [email protected]
> > Subject: [Opensymphony-developers] A couple of OSUser questions
> > 
> > 
> > Hi all,
> > 
> > Back from some extended time away from the keys.  Doing some 
> > travel and
> > carpentry and things of that nature.  It's fun to be back.  
> > Hope all your
> > holidays are going well.
> > 
> > Just getting recent versions of the code down and was psyched to see
> > Hibernate support in OSUser.  I've been working a bit with it 
> > in the last 24
> > hours and filed one bug on it last night.  I have the fixes 
> > for it, but it
> > will take longer for someone to use them than it will for 
> > someone with IDEA
> > and a RW connection to CVS to do the changes over again.  It 
> > took me about 30
> > minutes on this side.
> > 
> > The other one, I'm not so sure it's a bug or not, it smells a 
> > lot like it
> > belongs in the space of "conventions" rather than bug.  So I 
> > thought I would
> > ask for some rationale.  
> > 
> > In the CMP world, primary keys are objects instead of 
> > primitives.  I've been
> > following that through to Hibernate since it looks like 
> > composite types
> > require that as well.  But BaseHibernateEntity defines the PK 
> > as a primitive
> > long.  
> > 
> > Any comments on why this would be beneficial?  Otherwise, my 
> > vote is to
> > convert to object PKs and am happy to do the work on it.
> > 
> > thanks,
> > 
> > -b
> > 
> > 
> > -------------------------------------------------------
> > This SF.net email is sponsored by: SF.net Giveback Program.
> > Does SourceForge.net help you be more productive?  Does it
> > help you create better code?  SHARE THE LOVE, and help us help
> > YOU!  Click Here: http://sourceforge.net/donate/
> > _______________________________________________
> > Opensymphony-developers mailing list
> > [email protected]
> > https://lists.sourceforge.net/lists/listinfo/opensymphony-developers
> 


-------------------------------------------------------
This SF.net email is sponsored by: SF.net Giveback Program.
Does SourceForge.net help you be more productive?  Does it
help you create better code?  SHARE THE LOVE, and help us help
YOU!  Click Here: http://sourceforge.net/donate/