Deepal,

The question is how to best implement this (see AXIS2-4347). There are
two options:

1. Use an existing method of the AxisObserver interface and only
define a new event type. Disadvantage: We can only provide limited
information to the observer. Advantage: No modification of the API
required.

2. Define a new method in AxisObserver. Advantage: We can pass the
AxisModule and AxisDescription object to the method. Disadvantage:
This will break existing AxisObserver implementations (at build time,
not at runtime).

What is your opinion?

I recently worked with AxisObserver in the context of the transports
and I think that anyway we should add an AbstractAxisObserver (with
empty default implementations for the methods defined in AxisObserver)
and change the Javadoc of AxisObserver to recommend extending
AbstractAxisObserver instead of implementing AxisObserver directly.
This pattern makes sure that in the future we can add new methods to
AxisObserver without breaking anything.

Andreas

On Wed, May 13, 2009 at 15:28, Deepal jayasinghe <deep...@gmail.com> wrote:
> go for it.
>
> Deepal
>> Hi ,
>>
>> Currently AxisObserver does not get notified when a Module engaged or
>> disengaged in the Runtime.
>> So to have that behaviour i would like purpose to add  two Axis events
>> Named MODULE_ENGAGED , MODULE_DISENGAGED
>>
>> and in the new behaviour  when a module get engaged/disengaged to a
>> Service or to an Operation AxisObserver will get notified with
>> above Events.
>> So if there is no issues regarding this improvement i would like to
>> provide a patch to Axis2 trunk
>>
>> thank you,
>>
>> Charith Dhanushka Wickramarachchi
>> http://charithwiki.blogspot.com/
>>
>
>
> --
> Thank you!
>
>
> http://blogs.deepal.org
> http://deepal.org
>
>

Reply via email to