axis-java-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Afkham Azeez" <afk...@gmail.com>
Subject Re: [Axis2] Removing ContextListener
Date Fri, 01 Jun 2007 11:31:04 GMT
I'm removing this interface and the related code since, in our clustering
implementation, we are not going to send messages when contexts are added
anymore.

-- Azeez

On 5/29/07, Afkham Azeez <afkham@gmail.com> wrote:
>
> Hi Glen,
> OK got it. I'll integrate this once I finish what I'm working on right
> now.
>
> Thanks
> Azeez
>
> On 5/29/07, Glen Daniels < glen@thoughtcraft.com> wrote:
> >
> > Hi Afkham!
> >
> > Afkham Azeez wrote:
> > > Hi Glen,
> > > When I checked the logs, I found that you have introduced this
> > > interface. What is the usage of this interface?
> >
> > This was added because we refactored ContextFactory away - and the
> > reason for ContextFactory was to have one place that we could be
> > notified of all the *Context creations (even though it never actually
> > did that!).
> >
> > See thread at http://marc.info/?l=axis-dev&m=117822877408932&w=2 .
> > Apparently no one integrated this into the actual clustering code.
> >
> > > In the clustering implementation, we need to track the add context &
> > > remove context events for service group contexts & service contexts.
> > > Currently we are directly calling cluster manager from the relevant
> > > places. Can this interface be adopted by the clustering
> > implementation?
> >
> > Well, that was the idea... :)
> >
> > Do whatever the right thing is.  If you feel you're getting good mileage
> >
> > out of the current code and there isn't too much tight coupling, feel
> > free to remove ContextListener and associated APIs.  Otherwise feel free
> > to utilize it.
> >
> > Thanks,
> > --Glen
> >
> > > On 5/28/07, *Afkham Azeez* < afkham@gmail.com <mailto:afkham@gmail.com
> > >>
> > > wrote:
> > >
> > >     The ContextListener interface is not used by the clustering
> > >     implementation. We directly call ContextManager.addContext() from
> > >     the relevant places.
> > >     Is this interface intended for any other purpose? Shall we remove
> > it?
> >
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: axis-dev-unsubscribe@ws.apache.org
> > For additional commands, e-mail: axis-dev-help@ws.apache.org
> >
> >
>
>
> --
> Thanks
> Afkham Azeez
>
> http://www.wso2.org
> GPG Fingerprint: 643F C2AF EB78 F886 40C9  B2A2 4AE2 C887 665E 0760
>



-- 
Thanks
Afkham Azeez

http://www.wso2.org
GPG Fingerprint: 643F C2AF EB78 F886 40C9  B2A2 4AE2 C887 665E 0760

Mime
View raw message