Glen Daniels wrote:
Hi Deepal!

Deepal Jayasinghe wrote:
The main idea of introducing that was to provide module related data
from services . Say for an instance we engage module called "foo"
globally by putting <module ref="foo"> in axis2.xml . And let's say that
module to be work every service needs provide some kind of data , then
those data can be added in side that.

I see two situations here. In the first, the Module explicitly knows that it wants data from the active AxisService that is relevant to its operation. So the Module in this case should ensure that it runs after dispatch has occurred, and IMO it should just pull this configuration information out of the AxisService properties (mc.getService().getProperty("myModule.shoeSize") in the code and <service><parameter name="myModule.shoeSize"> in the XML).

+1; this is simply a choice of the module author which must be appropriately documented by them.

I think you meant <service><property name="..">.

In the second situation, the service wants to make sure the module is engaged and configured in a particular way, without the module caring whether it gets these params from the service or not. In this case, I don't see any reason it couldn't do:

<service>
 <module ref="myModule">
  <parameter name="shoeSize">

This assumes (reasonably, I think) that it's fine for a Service to engage a module that's already engaged globally. More generally, it's fine for any AxisDescription to engage a module that's already active at a higher level. The question comes up though whether they should be able to override the parameter settings - this is really a configuration choice, since in some situations you DO want to be able to override defaults, and in other situations the global deployer is setting site-wide policies that should NOT be changeable. Good use case for locked parameters....

The net is that I still don't see a need to introduce a separate <moduleConfig> structure, since we can handle the use-case you describe with either of the (simpler, I think) patterns above.

+1; seems cleaner to me.

Sanjiva.
--
Sanjiva Weerawarana, Ph.D.
Founder & Director; Lanka Software Foundation; http://www.opensource.lk/
Founder, Chairman & CEO; WSO2, Inc.; http://www.wso2.com/
Director; Open Source Initiative; http://www.opensource.org/
Member; Apache Software Foundation; http://www.apache.org/
Visiting Lecturer; University of Moratuwa; http://www.cse.mrt.ac.lk/

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to