Hi Rickard,

thanks for your comments which now let me see some things clearer and
hopefully
helped to restate my arguments as follows ...

-----Urspr�ngliche Nachricht-----
>Von: Rickard �berg [mailto:[EMAIL PROTECTED]]
>Gesendet: Mittwoch, 6. Dezember 2000 13:49
>An: jBoss Developer
>Betreff: Re: [jBoss-Dev] Proposed refactoring of ContainerFactory -
>ChainingDeployment se rvices in general

>Sounds a lot like our J2EE-deployer.

Yes, I love to paste our additional deployment services into the
J2EE-deployer. 

>This is not necessary. It works with the current ContainerFactory. All
>you have to do in your deployer (and which the J2EEDeployer does) is set
>the parent classloader as thread context classloader. If you look in
>deploy() it will be picked up as parent to the EJB classloader, hence
>keeping things coherent.

Now I see, but somehow don�t like it: 

- Its a rather implicit mechanism, isn�t it? Relies on the fact that each
deployer/service depending on class/classloader identity knows that the leaf
loaders are indeed redundant and checks for the parent for
comparison ... hmmm ... at least not what I would have implemented (or
expected any other RMI/ORB-implementations to have employed) as the default
optimisation check.

- Is IMHO not so clean and explicit as having the services communicating the
classloader 
explicitely via an additional deploy(ClassLoader, URL) method ...
 
- Makes it impossible to extend (i.e., build non-faked childs of) the leaf
classloaders coherently among several services on the fly (see my idea about
a big, modular system comprised out of individual, but interdependent
ejb-applications). 

>I can agree with a). Convince me of b).

In the end, you�ve cut down my arguments into two:

a) makes the ContainerFactory (maybe afterwards the J2EEDeployer,
EmbeddedTomcat) code more readable

b) allows for a reusability of J2EEDeployer, ContainerFactory and
EmbeddedTomcat also when having
"non-flat" deployment structures.  

Maybe b) is just a non-standard infor issue (we are not using Entity Beans
as data sources, but rather persistable and serializable Java-Beans that are
communicated through Session Beans and hence are also deployed within the
ejb-jars) ? 

> I'm ok with splitting up deploy(), but only for readability. The other
>reasons are (AFAICT) non-issues.

Best,
CGJ

Reply via email to