Yeah, that seems improves the performance. +1 for the change.
Chamikara
On 2/6/07, Rajith Attapattu <[EMAIL PROTECTED]> wrote:
Chamikara please see my reply to Bill questions.
As I mentioned I check and replicate state only at the end of an
invocation.
Thats why I changed the ClusterManager interface slightly.
Rajith Attapattu
Red Hat.
On 2/6/07, Chamikara Jayalath <[EMAIL PROTECTED]> wrote:
>
> Hi Bill,
>
> This will be called only when clustering is enabled.
>
> Even then we most probably will not be calling it in every property
> change. Rather will be trying to detect updates to the property bags in
> certain places of the execution.
>
> Plus better approaches are always welcome :-)
>
> Chamikara
>
>
> On 2/6/07, Bill Nagy <[EMAIL PROTECTED] > wrote:
> >
> > That's what I inferred from the interface -- any time there is a
> > context
> > change or a property change, you want your interface to be invoked.
> > That's an awful lot of method invocations (even if they're no-ops) for
> > something arguably not core to a web services runtime.
> >
> > -Bill
> >
> > On Tue, 2007-02-06 at 21:32 +0530, Chamikara Jayalath wrote:
> > > Hi Dims,
> > >
> > > As this is still is its design stage final approach hs not been
> > > decided. The current code is available in a branch so no affect on
> > the
> > > performance of the trunk.
> > >
> > > A method of the ClusterManager instance hs to be called in events
> > such
> > > as Context creation and removal. Currently these directly get called
> > > from respective places of the context classes (should happen only if
> > a
> > > ClusterManager hs been set i.e. when clustering is enabled).
> > >
> > > Chamikara
> > >
> > >
> > > On 2/6/07, Davanum Srinivas < [EMAIL PROTECTED]> wrote:
> > > Chamikara,
> > >
> > > New notifications? like what we have already for
> > > services/modules?
> > >
> > > -- dims
> > >
> > > On 2/6/07, Chamikara Jayalath < [EMAIL PROTECTED]>
> > wrote:
> > > > Hi Bill, Dims,
> > > >
> > > > The ClusterManager impl will be initiated with the
> > > ConfigContext. This will
> > > > be called in places like Context Creation and removal and
> > > may be property
> > > > updates (hvnt fully decided yet). When Clustering is not
> > > present the cost
> > > > should be reduced to a null check.
> > > >
> > > > Chamikara
> > > >
> > > >
> > > >
> > > >
> > > > On 2/6/07, Davanum Srinivas < [EMAIL PROTECTED]> wrote:
> > > > > Bill,
> > > > >
> > > > > from what i understand, it's in a separate maven module.
> >
> > > Not in
> > > > > kernel. So you can ignore it :)
> > > > >
> > > > > -- dims
> > > > >
> > > > > On 2/6/07, Bill Nagy < [EMAIL PROTECTED] > wrote:
> > > > > > What is the cost of this going to be for someone who
> > > doesn't want
> > > > > > clustering/this particular approach to clustering? I
> > > can't infer much
> > > > > > from the interface -- exactly which respective events
> > > are you referring
> > > > > > to?
> > > > > >
> > > > > > -Bill
> > > > > >
> > > > > >
> > > > > > On Thu, 2007-02-01 at 23:16 +0530, Chamikara Jayalath
> > > wrote:
> > > > > > > At initiation Axis2 will load a ClusterManager
> > > implementation object
> > > > > > > (if configured) and will call the methods of it
> > > whenever respective
> > > > > > > events occur.
> > > > > > >
> > > > > > > Chamikara
> > > > > > >
> > > > > > >
> > > > > > > [1] ClusterManager interface
> > > > > > >
> > > > > > > public abstract class ClusterManager {
> > > > > > >
> > > > > > > public abstract void init (ConfigurationContext
> > > context);
> > > > > > > public abstract void addContext (AbstractContext
> > > context);
> > > > > > > public abstract void removeContext
> > > (AbstractContext context);
> > > > > > > public abstract void addProperty
> > (AbstractContext
> > > context, String
> > > > > > > propertyName, Object propertyValue);
> > > > > > > public abstract void removeProperty
> > > (AbstractContext context,
> > > > > > > String propertyName);
> > > > > > > public abstract void touchProperty
> > > (AbstractContext context,
> > > > > > > String propertyName);
> > > > > > > public abstract void commit ();
> > > > > > >
> > > > > > > }
> > > > > > >
> > > > > >
> > > > > >
> > > > > >
> > > >
> > >
> > ---------------------------------------------------------------------
> > > > > > To unsubscribe, e-mail:
> > > > [EMAIL PROTECTED]
> > > > > > For additional commands, e-mail:
> > > [EMAIL PROTECTED]
> > > > > >
> > > > > >
> > > > >
> > > > >
> > > > > --
> > > > > Davanum Srinivas :: http://wso2.org/:: Oxygen for Web
> > > Services Developers
> > > > >
> > > > >
> > > >
> > >
> > ---------------------------------------------------------------------
> > > > > To unsubscribe, e-mail:
> > > > [EMAIL PROTECTED]
> > > > > For additional commands, e-mail:
> > > [EMAIL PROTECTED]
> > > > >
> > > > >
> > > >
> > > >
> > >
> > >
> > > --
> > > Davanum Srinivas :: http://wso2.org/ :: Oxygen for Web
> > > Services Developers
> > >
> > >
> > ---------------------------------------------------------------------
> > > To unsubscribe, e-mail: [EMAIL PROTECTED]
> > > For additional commands, e-mail: [EMAIL PROTECTED]
> > >
> > >
> >
> >
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: [EMAIL PROTECTED]
> > For additional commands, e-mail: [EMAIL PROTECTED]
> >
> >
>
