You are viewing a plain text version of this content. The canonical link for it is here.
Posted to java-dev@axis.apache.org by David Illsley <da...@gmail.com> on 2008/01/09 14:26:57 UTC
[Axis2] ConfigurationContext synchronization change sanity check
All,
I was looking at AXIS2-3283 which propted me to take a look at the
operationContextMap in ConfigurationContext and it struck me that we
can probably do better than the current synchronization blocks by
using a ConcurrentHashMap.
I 've attached a patch to make this change. Could someone take a look
and sanity check it please?
David
--
David Illsley - IBM Web Services Development
Re: [Axis2] ConfigurationContext synchronization change sanity check
Posted by David Illsley <da...@gmail.com>.
Thanks for taking a look. Unfortunately removing synchronization
blocks means indentation changes and a big patch :-(
David
On Jan 10, 2008 2:53 PM, Deepal Jayasinghe <de...@opensource.lk> wrote:
> hi David ,
> seems like a lot of changes :-) , any way if the build is working fine
> then no issues ;-)
>
> -Deepal
> > All,
> > I was looking at AXIS2-3283 which propted me to take a look at the
> > operationContextMap in ConfigurationContext and it struck me that we
> > can probably do better than the current synchronization blocks by
> > using a ConcurrentHashMap.
> > I 've attached a patch to make this change. Could someone take a look
> > and sanity check it please?
> >
> > David
> >
> > ------------------------------------------------------------------------
> >
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: axis-dev-unsubscribe@ws.apache.org
> > For additional commands, e-mail: axis-dev-help@ws.apache.org
>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: axis-dev-unsubscribe@ws.apache.org
> For additional commands, e-mail: axis-dev-help@ws.apache.org
>
>
--
David Illsley - IBM Web Services Development
---------------------------------------------------------------------
To unsubscribe, e-mail: axis-dev-unsubscribe@ws.apache.org
For additional commands, e-mail: axis-dev-help@ws.apache.org
Re: [Axis2] ConfigurationContext synchronization change sanity check
Posted by Deepal Jayasinghe <de...@opensource.lk>.
hi David ,
seems like a lot of changes :-) , any way if the build is working fine
then no issues ;-)
-Deepal
> All,
> I was looking at AXIS2-3283 which propted me to take a look at the
> operationContextMap in ConfigurationContext and it struck me that we
> can probably do better than the current synchronization blocks by
> using a ConcurrentHashMap.
> I 've attached a patch to make this change. Could someone take a look
> and sanity check it please?
>
> David
>
> ------------------------------------------------------------------------
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: axis-dev-unsubscribe@ws.apache.org
> For additional commands, e-mail: axis-dev-help@ws.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: axis-dev-unsubscribe@ws.apache.org
For additional commands, e-mail: axis-dev-help@ws.apache.org