Hi!

marc fleury wrote:
> Ok it is done for the entity stuff,
> sorry it took me time juha I had a hard concentrating.
> 
> 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
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.

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

Reply via email to