[GitHub] cxf pull request: Reducing synchronized block in ServletController...

2014-12-19 Thread lucaspouzac
GitHub user lucaspouzac opened a pull request:

https://github.com/apache/cxf/pull/45

Reducing synchronized block in ServletController class

Hi, 

When I run performance test, I see a contention in ServletController.

I think the synchronization block can be reduced as this PR.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/lucaspouzac/cxf 3.0.x-fixes

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cxf/pull/45.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #45


commit 7b000478135d1679def4b0d5317244a8f20bc5d4
Author: Lucas Pouzac lpouzac.exte...@pagesjaunes.fr
Date:   2014-12-19T10:58:48Z

Reducing synchronized block in ServletController class




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cxf pull request: Reducing synchronized block in ServletController...

2014-12-19 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/cxf/pull/45


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cxf pull request: Reducing synchronized block in ServletController...

2014-12-19 Thread sberyozkin
Github user sberyozkin commented on the pull request:

https://github.com/apache/cxf/pull/45#issuecomment-67645463
  
Hi, thanks for the patch, I've applied it with minor modifications (keep 
the sync block on if a  disableAddressProperties is set to a legacy false value)
Cheers, Sergey


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---