> > Please review the changes it is pretty trivial and I will put
> it in the find
> > methods as well :)
>
> -1. Doing these calls in EntityContainer won't work. Motivation:
> * There is a difference in semantics of these calls between BMP and CMP.
> The only part that knows if we're doing BMP or CMP is the persistence
> manager. Hence the persistence manager must make these calls. The
wrong.
exposing CMP or BMP with metadata is trivial really.
And then we can do the right thing re: implementing EJB calls in the right
place.
Don't make a big architectural decision such as "all ejb calls will be
called by the persistence engine" based on the fact you don't put the
relevant information in the right place.
> persistence manager also makes all the other calls so this is nothing
> new.
> * EntityContainer is glue only. It should not do any actual work. All it
> does is delegate to other parts to do the actual work.
wow, JAWSPersitenceManager.java is "container" but EntityContainer.java is
"glue"?????
wow, you can't be serious.
BTW, before talking about the create stuff we are debugging (still doesn't
work ;-) maybe fixing the find bug would be more productive.
marc
> I saw that you have already made the change, so please change it back.
>
> /Rickard
>
> --
> Rickard �berg
>
> @home: +46 13 177937
> Email: [EMAIL PROTECTED]
> http://www.telkel.com
> http://www.jboss.org
> http://www.dreambean.com
>
>
