Re: Axis 2 Memory Leak when Client is instantiated within Service - pertains to edu.emory...ConcurrentHashMap - WAS: Re: When do cleanup functions need to be called?

2009-06-03 Thread jijisv

Matt,

did you get this issue fixed?
I am facing the same error in the project I am working on. I am using Axis
1.4.1.
Appreciate your help.

Regards
Jiji


Matt Friedman wrote:
 
 Hi,
 
 I'm sorry for the re-post, but I was discussing this thread with a
 co-worker and we postulated that the title of the existing post (Re:
 When do cleanup functions need to be called?), didn't represent what
 this thread is actually about.
 
 I was afraid the importance of the thread might be missed, as the old
 title no longer applies.
 
 Hopefully, this isn't inappropriate. Sorry if I've annoyed anyone.
 
 So, renamed more appropriately, here is the thread...
 
 Many thanks,
 Matt Friedman.
 
 
 2008/9/23 Matt Friedman matt.fried...@gmail.com:
 Hi David,

 Thanks very much for this.

 When you say: Try 1.4.1. It's much better than 1.4 with this
 particular memory leak., I take it you mean it is partially fixed,
 but not entirely fixed in 1.4.1. Is that correct?

 Did your pool of stubs continue to work for you?

 My goal is to completely eliminate any mem leaks from our app.

 Thanks,
 Matt.


 2008/9/23 David Rees dree...@gmail.com:
 On Mon, Sep 22, 2008 at 6:39 PM, Matt Friedman matt.fried...@gmail.com
 wrote:
 Hi, was this issue fixed or resolved? I am running our service in
 JProbe and we are having the identical memory leak issues as described
 in this email chain. After roughly 900 requests the garbage collector
 goes to 100% and out of memory occurs.

 Over half the heap is full with instances related to:
 edu.emory.mathcs.backport.java.util.concurrent.ConcurrentHashMap.*
 which is tied to the Axis2 implementation. We don't use edu.emory.* at
 all.

 We are running Axis2 v1.4. We're going to try a more recent release or
 the svn nightly build.

 Try 1.4.1. It's much better than 1.4 with this particular memory leak.
 If your access to the stub constructor isn't synchronized, you may
 need to serialize access to stub constructors as well.

 -Dave

 -
 To unsubscribe, e-mail: axis-user-unsubscr...@ws.apache.org
 For additional commands, e-mail: axis-user-h...@ws.apache.org





 --
 -- Matt Friedman

 
 
 
 -- 
 -- Matt Friedman
 
 -
 To unsubscribe, e-mail: axis-user-unsubscr...@ws.apache.org
 For additional commands, e-mail: axis-user-h...@ws.apache.org
 
 
 

-- 
View this message in context: 
http://www.nabble.com/Axis-2-Memory-Leak-when-Client-is-instantiated-within-Service---pertains-to-edu.emory...ConcurrentHashMap--%3E-WAS%3A-Re%3A-When-do-cleanup-functions-need-to-be-called--tp19631299p23850641.html
Sent from the Axis - User mailing list archive at Nabble.com.



Re: When do cleanup functions need to be called?

2008-11-29 Thread Mel T



Amila Suriarachchi wrote:
 
 Please try with a Nighly build. As I think this could be the problem.
 ...
 -- 
 Amila Suriarachchi
 WSO2 Inc.
 blog: http://amilachinthaka.blogspot.com/
 

Any idea when an release with this fix is planned ? Would this be in  1.4.2
?

-- 
View this message in context: 
http://www.nabble.com/When-do-cleanup-functions-need-to-be-called--tp18724207p20749331.html
Sent from the Axis - User mailing list archive at Nabble.com.


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Re: When do cleanup functions need to be called?

2008-10-07 Thread Amila Suriarachchi
Please try with a Nighly build. As I think this could be the problem.

in Stub class we have this at the clean up method.

protected void finalize() throws Throwable {
super.finalize();
cleanup();
}

   public void cleanup() throws AxisFault {
_service.getAxisConfiguration().removeService(_service.getName());
}

this removes the service from the Axisconfiguration but does not remove the
end points.

If we take the ServiceClient clean up method. it calls to
axisConfiguration.removeServiceGroup(serviceGroupName);

this method looks like this,

Iterator services = axisServiceGroup.getServices();
while (services.hasNext()) {
AxisService axisService = (AxisService) services.next();
allServices.remove(axisService.getName());
if (!axisService.isClientSide()) {
notifyObservers(AxisEvent.SERVICE_REMOVE, axisService);
}

//removes the endpoints to this service
String serviceName = axisService.getName();
String key = null;
for (Iterator iter =
axisService.getEndpoints().keySet().iterator(); iter.hasNext();){
key = serviceName + . + (String)iter.next();
this.allEndpoints.remove(key);
}

}

As you can see here Now the service is not available and as a result of that
endpoints are not removed.

I removed the service removal code from the Stub and as a result of that I
did see the memory leak with was there earlier.

Axis2 trunk has this change already.

thanks,
Amila.



On Tue, Oct 7, 2008 at 7:35 AM, Mel T [EMAIL PROTECTED] wrote:




  I have the same problem as described here.
  Upgrading to 1.4.1 has reduced the leak.
  I am now going to try the pool of stubs as well for the remaining.
 
  Dave, is your client running in Tomcat ?
  My problem only occurs when I am accessing the WS using a stub from
  within
  the same tomcat application.
 
  Yes, my client is running in Tomcat, Tomcat 5.5.27 to be exact. I
  wonder if that is somehow also contributing to the issue?
 
  -Dave
 

 I believe this is an issue with that setup. Not easy to debug. I will post
 here if I find something.

 --
 View this message in context:
 http://www.nabble.com/When-do-cleanup-functions-need-to-be-called--tp18724207p19849777.html
 Sent from the Axis - User mailing list archive at Nabble.com.


 -
 To unsubscribe, e-mail: [EMAIL PROTECTED]
 For additional commands, e-mail: [EMAIL PROTECTED]




-- 
Amila Suriarachchi
WSO2 Inc.
blog: http://amilachinthaka.blogspot.com/


Re: When do cleanup functions need to be called?

2008-10-06 Thread Mel T


I have the same problem as described here. 
Upgrading to 1.4.1 has reduced the leak.
I am now going to try the pool of stubs as well for the remaining.

Dave, is your client running in Tomcat ?
My problem only occurs when I am accessing the WS using a stub from within
the same tomcat application.

--
Mel T.
-- 
View this message in context: 
http://www.nabble.com/When-do-cleanup-functions-need-to-be-called--tp18724207p19848906.html
Sent from the Axis - User mailing list archive at Nabble.com.


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Re: When do cleanup functions need to be called?

2008-10-06 Thread David Rees
On Mon, Oct 6, 2008 at 5:27 PM, Mel T [EMAIL PROTECTED] wrote:
 I have the same problem as described here.
 Upgrading to 1.4.1 has reduced the leak.
 I am now going to try the pool of stubs as well for the remaining.

 Dave, is your client running in Tomcat ?
 My problem only occurs when I am accessing the WS using a stub from within
 the same tomcat application.

Yes, my client is running in Tomcat, Tomcat 5.5.27 to be exact. I
wonder if that is somehow also contributing to the issue?

-Dave

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Re: When do cleanup functions need to be called?

2008-10-06 Thread Mel T



 I have the same problem as described here.
 Upgrading to 1.4.1 has reduced the leak.
 I am now going to try the pool of stubs as well for the remaining.

 Dave, is your client running in Tomcat ?
 My problem only occurs when I am accessing the WS using a stub from
 within
 the same tomcat application.
 
 Yes, my client is running in Tomcat, Tomcat 5.5.27 to be exact. I
 wonder if that is somehow also contributing to the issue?
 
 -Dave
 

I believe this is an issue with that setup. Not easy to debug. I will post
here if I find something.

-- 
View this message in context: 
http://www.nabble.com/When-do-cleanup-functions-need-to-be-called--tp18724207p19849777.html
Sent from the Axis - User mailing list archive at Nabble.com.


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Re: When do cleanup functions need to be called?

2008-09-23 Thread Matt Friedman
Hi David,

Thanks very much for this.

When you say: Try 1.4.1. It's much better than 1.4 with this
particular memory leak., I take it you mean it is partially fixed,
but not entirely fixed in 1.4.1. Is that correct?

Did your pool of stubs continue to work for you?

My goal is to completely eliminate any mem leaks from our app.

Thanks,
Matt.


2008/9/23 David Rees [EMAIL PROTECTED]:
 On Mon, Sep 22, 2008 at 6:39 PM, Matt Friedman [EMAIL PROTECTED] wrote:
 Hi, was this issue fixed or resolved? I am running our service in
 JProbe and we are having the identical memory leak issues as described
 in this email chain. After roughly 900 requests the garbage collector
 goes to 100% and out of memory occurs.

 Over half the heap is full with instances related to:
 edu.emory.mathcs.backport.java.util.concurrent.ConcurrentHashMap.*
 which is tied to the Axis2 implementation. We don't use edu.emory.* at
 all.

 We are running Axis2 v1.4. We're going to try a more recent release or
 the svn nightly build.

 Try 1.4.1. It's much better than 1.4 with this particular memory leak.
 If your access to the stub constructor isn't synchronized, you may
 need to serialize access to stub constructors as well.

 -Dave

 -
 To unsubscribe, e-mail: [EMAIL PROTECTED]
 For additional commands, e-mail: [EMAIL PROTECTED]





-- 
-- Matt Friedman

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Axis 2 Memory Leak when Client is instantiated within Service - pertains to edu.emory...ConcurrentHashMap - WAS: Re: When do cleanup functions need to be called?

2008-09-23 Thread Matt Friedman
Hi,

I'm sorry for the re-post, but I was discussing this thread with a
co-worker and we postulated that the title of the existing post (Re:
When do cleanup functions need to be called?), didn't represent what
this thread is actually about.

I was afraid the importance of the thread might be missed, as the old
title no longer applies.

Hopefully, this isn't inappropriate. Sorry if I've annoyed anyone.

So, renamed more appropriately, here is the thread...

Many thanks,
Matt Friedman.


2008/9/23 Matt Friedman [EMAIL PROTECTED]:
 Hi David,

 Thanks very much for this.

 When you say: Try 1.4.1. It's much better than 1.4 with this
 particular memory leak., I take it you mean it is partially fixed,
 but not entirely fixed in 1.4.1. Is that correct?

 Did your pool of stubs continue to work for you?

 My goal is to completely eliminate any mem leaks from our app.

 Thanks,
 Matt.


 2008/9/23 David Rees [EMAIL PROTECTED]:
 On Mon, Sep 22, 2008 at 6:39 PM, Matt Friedman [EMAIL PROTECTED] wrote:
 Hi, was this issue fixed or resolved? I am running our service in
 JProbe and we are having the identical memory leak issues as described
 in this email chain. After roughly 900 requests the garbage collector
 goes to 100% and out of memory occurs.

 Over half the heap is full with instances related to:
 edu.emory.mathcs.backport.java.util.concurrent.ConcurrentHashMap.*
 which is tied to the Axis2 implementation. We don't use edu.emory.* at
 all.

 We are running Axis2 v1.4. We're going to try a more recent release or
 the svn nightly build.

 Try 1.4.1. It's much better than 1.4 with this particular memory leak.
 If your access to the stub constructor isn't synchronized, you may
 need to serialize access to stub constructors as well.

 -Dave

 -
 To unsubscribe, e-mail: [EMAIL PROTECTED]
 For additional commands, e-mail: [EMAIL PROTECTED]





 --
 -- Matt Friedman




-- 
-- Matt Friedman

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Re: Axis 2 Memory Leak when Client is instantiated within Service - pertains to edu.emory...ConcurrentHashMap - WAS: Re: When do cleanup functions need to be called?

2008-09-23 Thread Matt Friedman
I'm really sorry, I didn't attach the whole thread the first time.
Here are the details...
--




Hi, was this issue fixed or resolved? I am running our service in
JProbe and we are having the identical memory leak issues as described
in this email chain. After roughly 900 requests the garbage collector
goes to 100% and out of memory occurs.

Over half the heap is full with instances related to:
edu.emory.mathcs.backport.java.util.concurrent.ConcurrentHashMap.*
which is tied to the Axis2 implementation. We don't use edu.emory.* at
all.

We are running Axis2 v1.4. We're going to try a more recent release or
the svn nightly build.

Any thoughts or recommendations?

If you need any further information please let me know.

Many thanks,
Matt Friedman.



2008/7/31 Amila Suriarachchi [EMAIL PROTECTED]:
- Hide quoted text -


 On Wed, Jul 30, 2008 at 11:08 PM, David Rees [EMAIL PROTECTED] wrote:

 On Wed, Jul 30, 2008 at 4:22 AM, Amila Suriarachchi
 [EMAIL PROTECTED] wrote:
  On Wed, Jul 30, 2008 at 2:15 PM, David Rees [EMAIL PROTECTED] wrote:
 
  well see this code,
 
  public static void clientCall1() {
   MyServiceStub stub = new MyServiceStub(http://example.com/myservice;);
   ClientCall1 req = new ClientCall1();
   ClientCall1Response res = stub.clientCall1(req);
   // Need to call this when calling using this function in a web service
   // Otherwise huge resource leak occurs
   stub._getServiceClient().
  cleanup();
   }
  }
 
  After this method call this class should be garbage collected. since
  stub
  does not have any reference after that.

 Right - but I'm not having a problem with Stubs being leaked. Here's a
 count of various interesting classes (I've skipped some of the Java
 core classes for brevity) of which there are a high number of
 instances when the heap runs out of space (using a 32mb heap):

 edu.emory.mathcs.backport.java.util.concurrent.ConcurrentHashMap$Segment
 177,023, 17% heap

 edu.emory.mathcs.backport.java.util.concurrent.ConcurrentHashMap$HashEntry[]
 177,023, 6% heap

 edu.emory.mathcs.backport.java.util.concurrent.locks.ReentrantLock$NonfairSync
 177,023, 8% heap
 java.util.HashMap$Entry[] 37926, 8% heap
 java.util.HashMap 36789, 4% heap
 java.util.HashMap$Entry 28853, 2% heap
 java.util.ArrayList 13698, 1% heap
 org.apache.axis2.description.ParameterIncludeImpl 11099, 0% heap
 edu.emory.mathcs.backport.java.util.concurrent.ConcurrentHashMap 11064, 1%
 heap
 org.apache.axis2.description.PolicySubject 11063, 0% heap
 org.apache.axis2.engine.Phase 7905, 0% heap
 org.apache.axis2.description.AxisBindingMessage 4668, 1% heap
 org.apache.axis2.description.AxisMessage 3168

 So it's pretty easy to see where the majority of the heap is being
 used, about half the heap is being used by the ConcurrentHashMap and
 HashMaps.

 Tracing a ConcurrentHashMap$Segment to it's root goes like this (class
 name w/variable name):

 ConcurrentHashMap$Segment
 ConcurrentHashMap$Segment[]
 ConcurrentHashMap (children)
 OutInAxisOperation
 HashMap$Entry
 HashMap$Entry[]
 HashMap operationsAliasesMap
 AxisService
 HashMap$Entry (value)
 HashMap$Entry[]
 HashMap (allEndpoints)

 this is the place this issue was fixed.

 AxisConfiguration (axisConfiguration)
 AxisServlet

 Hopefully that provides some insight as to where reference chain is going.

 Now, after more trial and error, I believe that I've figured out a way
 to keep it from leaking (the service has been running overnight now
 with a 32MB heap and it has not gone OOM yet):

 1. Use Axis2 1.4.1 RC1
 2. Create a pool of stubs to use so that only one thread uses a Stub at a
 time.

 #2 I tried after a lot of googling and mail archive searches where I
 found a post that indicated that the creation of and use of Stubs may
 not be thread safe. I don't know enough about the internals of axis2
 as to why this may be an issue, but hopefully it can point the
 developers in the right direction (still haven't been able to create a
 small test case).

 then we can have a better look.

 thanks,
 Amila.


 Thanks

 -Dave

 -
 To unsubscribe, e-mail: [EMAIL PROTECTED]
 For additional commands, e-mail: [EMAIL PROTECTED]




 --
 Amila Suriarachchi,
 WSO2 Inc.


2008/9/23 Matt Friedman [EMAIL PROTECTED]:
 Hi,

 I'm sorry for the re-post, but I was discussing this thread with a
 co-worker and we postulated that the title of the existing post (Re:
 When do cleanup functions need to be called?), didn't represent what
 this thread is actually about.

 I was afraid the importance of the thread might be missed, as the old
 title no longer applies.

 Hopefully, this isn't inappropriate. Sorry if I've annoyed anyone.

 So, renamed more appropriately, here is the thread...

 Many thanks,
 Matt Friedman.


 2008/9/23 Matt Friedman [EMAIL PROTECTED]:
 Hi David,

 Thanks very much for this.

 When you say: Try 1.4.1. It's much better than 1.4

Re: When do cleanup functions need to be called?

2008-09-23 Thread David Rees
On Tue, Sep 23, 2008 at 5:16 AM, Matt Friedman [EMAIL PROTECTED] wrote:
 When you say: Try 1.4.1. It's much better than 1.4 with this
 particular memory leak., I take it you mean it is partially fixed,
 but not entirely fixed in 1.4.1. Is that correct?

 Did your pool of stubs continue to work for you?

 My goal is to completely eliminate any mem leaks from our app.

Yes, it has continued to work for me, and I don't know of any memory leaks.

-Dave

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Re: When do cleanup functions need to be called?

2008-09-22 Thread Matt Friedman
Hi, was this issue fixed or resolved? I am running our service in
JProbe and we are having the identical memory leak issues as described
in this email chain. After roughly 900 requests the garbage collector
goes to 100% and out of memory occurs.

Over half the heap is full with instances related to:
edu.emory.mathcs.backport.java.util.concurrent.ConcurrentHashMap.*
which is tied to the Axis2 implementation. We don't use edu.emory.* at
all.

We are running Axis2 v1.4. We're going to try a more recent release or
the svn nightly build.

Any thoughts or recommendations?

If you need any further information please let me know.

Many thanks,
Matt Friedman.



2008/7/31 Amila Suriarachchi [EMAIL PROTECTED]:


 On Wed, Jul 30, 2008 at 11:08 PM, David Rees [EMAIL PROTECTED] wrote:

 On Wed, Jul 30, 2008 at 4:22 AM, Amila Suriarachchi
 [EMAIL PROTECTED] wrote:
  On Wed, Jul 30, 2008 at 2:15 PM, David Rees [EMAIL PROTECTED] wrote:
 
  well see this code,
 
  public static void clientCall1() {
   MyServiceStub stub = new MyServiceStub(http://example.com/myservice;);
   ClientCall1 req = new ClientCall1();
   ClientCall1Response res = stub.clientCall1(req);
   // Need to call this when calling using this function in a web service
   // Otherwise huge resource leak occurs
   stub._getServiceClient().
  cleanup();
   }
  }
 
  After this method call this class should be garbage collected. since
  stub
  does not have any reference after that.

 Right - but I'm not having a problem with Stubs being leaked. Here's a
 count of various interesting classes (I've skipped some of the Java
 core classes for brevity) of which there are a high number of
 instances when the heap runs out of space (using a 32mb heap):

 edu.emory.mathcs.backport.java.util.concurrent.ConcurrentHashMap$Segment
 177,023, 17% heap

 edu.emory.mathcs.backport.java.util.concurrent.ConcurrentHashMap$HashEntry[]
 177,023, 6% heap

 edu.emory.mathcs.backport.java.util.concurrent.locks.ReentrantLock$NonfairSync
 177,023, 8% heap
 java.util.HashMap$Entry[] 37926, 8% heap
 java.util.HashMap 36789, 4% heap
 java.util.HashMap$Entry 28853, 2% heap
 java.util.ArrayList 13698, 1% heap
 org.apache.axis2.description.ParameterIncludeImpl 11099, 0% heap
 edu.emory.mathcs.backport.java.util.concurrent.ConcurrentHashMap 11064, 1%
 heap
 org.apache.axis2.description.PolicySubject 11063, 0% heap
 org.apache.axis2.engine.Phase 7905, 0% heap
 org.apache.axis2.description.AxisBindingMessage 4668, 1% heap
 org.apache.axis2.description.AxisMessage 3168

 So it's pretty easy to see where the majority of the heap is being
 used, about half the heap is being used by the ConcurrentHashMap and
 HashMaps.

 Tracing a ConcurrentHashMap$Segment to it's root goes like this (class
 name w/variable name):

 ConcurrentHashMap$Segment
 ConcurrentHashMap$Segment[]
 ConcurrentHashMap (children)
 OutInAxisOperation
 HashMap$Entry
 HashMap$Entry[]
 HashMap operationsAliasesMap
 AxisService
 HashMap$Entry (value)
 HashMap$Entry[]
 HashMap (allEndpoints)

 this is the place this issue was fixed.

 AxisConfiguration (axisConfiguration)
 AxisServlet

 Hopefully that provides some insight as to where reference chain is going.

 Now, after more trial and error, I believe that I've figured out a way
 to keep it from leaking (the service has been running overnight now
 with a 32MB heap and it has not gone OOM yet):

 1. Use Axis2 1.4.1 RC1
 2. Create a pool of stubs to use so that only one thread uses a Stub at a
 time.

 #2 I tried after a lot of googling and mail archive searches where I
 found a post that indicated that the creation of and use of Stubs may
 not be thread safe. I don't know enough about the internals of axis2
 as to why this may be an issue, but hopefully it can point the
 developers in the right direction (still haven't been able to create a
 small test case).

 then we can have a better look.

 thanks,
 Amila.


 Thanks

 -Dave

 -
 To unsubscribe, e-mail: [EMAIL PROTECTED]
 For additional commands, e-mail: [EMAIL PROTECTED]




 --
 Amila Suriarachchi,
 WSO2 Inc.




-- 
-- Matt Friedman

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Re: When do cleanup functions need to be called?

2008-07-30 Thread David Rees
On Tue, Jul 29, 2008 at 10:52 PM, Amila Suriarachchi
[EMAIL PROTECTED] wrote:
 this method should be call when the MyServiceStub (see finalize method in
 ServiceClient) is garbage collected. I am not sure why it is not being
 called automatically.

I don't think that it's being garbage collected at all. From what I
can tell, a bunch of objects are still being referenced by the
AxisConfiguration referenced by the AxisServlet.

 In RC1 we fixed a memory leak regarding this. But I can not realy think the
 reason for your problem.

Yes, 1.4.1 RC1 makes the situation a lot better than 1.4 - Calling
cleanup has no effect on the memory leak (see my earlier followup
email).

 you can get the AxisConfiguration object like this,
 AxisConfiguration axiConfiguration =
 MessageContext.getCurrentMessageContext().getConfigurationContext().getAxisConfiguration();

 and check whether there are some additional objects there after calling
 clean up.

 Please log a jira.

Once I can create a small reproducable test-case I'll log a jira issue.

Let me know if there is any other information I can gather or if
there's anything else you want me to try in the meantime.

Thanks,

-Dave

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Re: When do cleanup functions need to be called?

2008-07-30 Thread Pierre Muller




On spec, wouldn't it be related to the scope of your services ?
http://www.developer.com/java/web/article.php/3620661


David Rees a crit:

  On Tue, Jul 29, 2008 at 10:52 PM, Amila Suriarachchi
[EMAIL PROTECTED] wrote:
  
  
this method should be call when the MyServiceStub (see finalize method in
ServiceClient) is garbage collected. I am not sure why it is not being
called automatically.

  
  
I don't think that it's being garbage collected at all. From what I
can tell, a bunch of objects are still being referenced by the
AxisConfiguration referenced by the AxisServlet.

  
  
In RC1 we fixed a memory leak regarding this. But I can not realy think the
reason for your problem.

  
  
Yes, 1.4.1 RC1 makes the situation a lot better than 1.4 - Calling
cleanup has no effect on the memory leak (see my earlier followup
email).

  
  
you can get the AxisConfiguration object like this,
AxisConfiguration axiConfiguration =
MessageContext.getCurrentMessageContext().getConfigurationContext().getAxisConfiguration();

and check whether there are some additional objects there after calling
clean up.

Please log a jira.

  
  
Once I can create a small reproducable test-case I'll log a jira issue.

Let me know if there is any other information I can gather or if
there's anything else you want me to try in the meantime.

Thanks,

-Dave

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



  

**
Si vous n'êtes pas le destinataire désigné de ce message ou une personne autorisée à l'utiliser, toute distribution, copie, publication ou usage à quelques fins que ce soit des informations dans ce message sont interdits. Merci d'informer immédiatement l'expéditeur par messagerie, et, de détruire ce message.
This e-mail is confidential. If you are not the addressee or an authorized recipient of this message, any distribution, copying, publication or use of this information for any purpose is prohibited. Please notify the sender immediately by e-mail and then delete this message.
**






-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Re: When do cleanup functions need to be called?

2008-07-30 Thread Amila Suriarachchi
On Wed, Jul 30, 2008 at 2:15 PM, David Rees [EMAIL PROTECTED] wrote:

 On Tue, Jul 29, 2008 at 10:52 PM, Amila Suriarachchi
 [EMAIL PROTECTED] wrote:
  this method should be call when the MyServiceStub (see finalize method in
  ServiceClient) is garbage collected. I am not sure why it is not being
  called automatically.

 I don't think that it's being garbage collected at all. From what I
 can tell, a bunch of objects are still being referenced by the
 AxisConfiguration referenced by the AxisServlet.


well see this code,

public static void clientCall1() {
 MyServiceStub stub = new MyServiceStub(http://example.com/myservice;);
 ClientCall1 req = new ClientCall1();
 ClientCall1Response res = stub.clientCall1(req);
 // Need to call this when calling using this function in a web service
 // Otherwise huge resource leak occurs
 stub._getServiceClient().cleanup();
 }
}

After this method call this class should be garbage collected. since stub
does not have any reference after that.








  In RC1 we fixed a memory leak regarding this. But I can not realy think
 the
  reason for your problem.

 Yes, 1.4.1 RC1 makes the situation a lot better than 1.4 - Calling
 cleanup has no effect on the memory leak (see my earlier followup
 email).

  you can get the AxisConfiguration object like this,
  AxisConfiguration axiConfiguration =
 
 MessageContext.getCurrentMessageContext().getConfigurationContext().getAxisConfiguration();
 
  and check whether there are some additional objects there after calling
  clean up.
 
  Please log a jira.

 Once I can create a small reproducable test-case I'll log a jira issue.


yep. this helps a lot.

thanks,
Amila.



 Let me know if there is any other information I can gather or if
 there's anything else you want me to try in the meantime.

 Thanks,

 -Dave

 -
 To unsubscribe, e-mail: [EMAIL PROTECTED]
 For additional commands, e-mail: [EMAIL PROTECTED]




-- 
Amila Suriarachchi,
WSO2 Inc.


Re: When do cleanup functions need to be called?

2008-07-30 Thread David Rees
On Wed, Jul 30, 2008 at 4:22 AM, Amila Suriarachchi
[EMAIL PROTECTED] wrote:
 On Wed, Jul 30, 2008 at 2:15 PM, David Rees [EMAIL PROTECTED] wrote:

 well see this code,

 public static void clientCall1() {
  MyServiceStub stub = new MyServiceStub(http://example.com/myservice;);
  ClientCall1 req = new ClientCall1();
  ClientCall1Response res = stub.clientCall1(req);
  // Need to call this when calling using this function in a web service
  // Otherwise huge resource leak occurs
  stub._getServiceClient().
 cleanup();
  }
 }

 After this method call this class should be garbage collected. since stub
 does not have any reference after that.

Right - but I'm not having a problem with Stubs being leaked. Here's a
count of various interesting classes (I've skipped some of the Java
core classes for brevity) of which there are a high number of
instances when the heap runs out of space (using a 32mb heap):

edu.emory.mathcs.backport.java.util.concurrent.ConcurrentHashMap$Segment
177,023, 17% heap
edu.emory.mathcs.backport.java.util.concurrent.ConcurrentHashMap$HashEntry[]
177,023, 6% heap
edu.emory.mathcs.backport.java.util.concurrent.locks.ReentrantLock$NonfairSync
177,023, 8% heap
java.util.HashMap$Entry[] 37926, 8% heap
java.util.HashMap 36789, 4% heap
java.util.HashMap$Entry 28853, 2% heap
java.util.ArrayList 13698, 1% heap
org.apache.axis2.description.ParameterIncludeImpl 11099, 0% heap
edu.emory.mathcs.backport.java.util.concurrent.ConcurrentHashMap 11064, 1% heap
org.apache.axis2.description.PolicySubject 11063, 0% heap
org.apache.axis2.engine.Phase 7905, 0% heap
org.apache.axis2.description.AxisBindingMessage 4668, 1% heap
org.apache.axis2.description.AxisMessage 3168

So it's pretty easy to see where the majority of the heap is being
used, about half the heap is being used by the ConcurrentHashMap and
HashMaps.

Tracing a ConcurrentHashMap$Segment to it's root goes like this (class
name w/variable name):

ConcurrentHashMap$Segment
ConcurrentHashMap$Segment[]
ConcurrentHashMap (children)
OutInAxisOperation
HashMap$Entry
HashMap$Entry[]
HashMap operationsAliasesMap
AxisService
HashMap$Entry (value)
HashMap$Entry[]
HashMap (allEndpoints)
AxisConfiguration (axisConfiguration)
AxisServlet

Hopefully that provides some insight as to where reference chain is going.

Now, after more trial and error, I believe that I've figured out a way
to keep it from leaking (the service has been running overnight now
with a 32MB heap and it has not gone OOM yet):

1. Use Axis2 1.4.1 RC1
2. Create a pool of stubs to use so that only one thread uses a Stub at a time.

#2 I tried after a lot of googling and mail archive searches where I
found a post that indicated that the creation of and use of Stubs may
not be thread safe. I don't know enough about the internals of axis2
as to why this may be an issue, but hopefully it can point the
developers in the right direction (still haven't been able to create a
small test case).

Thanks

-Dave

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Re: When do cleanup functions need to be called?

2008-07-30 Thread David Rees
On Wed, Jul 30, 2008 at 2:09 AM, Pierre Muller [EMAIL PROTECTED] wrote:
 On spec, wouldn't it be related to the scope of your services ?
 http://www.developer.com/java/web/article.php/3620661

I don't think so - The default scope (request) should be fine. I don't
care about anything longer than that.

-Dave

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Re: When do cleanup functions need to be called?

2008-07-30 Thread Amila Suriarachchi
On Wed, Jul 30, 2008 at 11:08 PM, David Rees [EMAIL PROTECTED] wrote:

 On Wed, Jul 30, 2008 at 4:22 AM, Amila Suriarachchi
 [EMAIL PROTECTED] wrote:
  On Wed, Jul 30, 2008 at 2:15 PM, David Rees [EMAIL PROTECTED] wrote:
 
  well see this code,
 
  public static void clientCall1() {
   MyServiceStub stub = new MyServiceStub(http://example.com/myservice;);
   ClientCall1 req = new ClientCall1();
   ClientCall1Response res = stub.clientCall1(req);
   // Need to call this when calling using this function in a web service
   // Otherwise huge resource leak occurs
   stub._getServiceClient().
  cleanup();
   }
  }
 
  After this method call this class should be garbage collected. since stub
  does not have any reference after that.

 Right - but I'm not having a problem with Stubs being leaked. Here's a
 count of various interesting classes (I've skipped some of the Java
 core classes for brevity) of which there are a high number of
 instances when the heap runs out of space (using a 32mb heap):

 edu.emory.mathcs.backport.java.util.concurrent.ConcurrentHashMap$Segment
 177,023, 17% heap

 edu.emory.mathcs.backport.java.util.concurrent.ConcurrentHashMap$HashEntry[]
 177,023, 6% heap

 edu.emory.mathcs.backport.java.util.concurrent.locks.ReentrantLock$NonfairSync
 177,023, 8% heap
 java.util.HashMap$Entry[] 37926, 8% heap
 java.util.HashMap 36789, 4% heap
 java.util.HashMap$Entry 28853, 2% heap
 java.util.ArrayList 13698, 1% heap
 org.apache.axis2.description.ParameterIncludeImpl 11099, 0% heap
 edu.emory.mathcs.backport.java.util.concurrent.ConcurrentHashMap 11064, 1%
 heap
 org.apache.axis2.description.PolicySubject 11063, 0% heap
 org.apache.axis2.engine.Phase 7905, 0% heap
 org.apache.axis2.description.AxisBindingMessage 4668, 1% heap
 org.apache.axis2.description.AxisMessage 3168

 So it's pretty easy to see where the majority of the heap is being
 used, about half the heap is being used by the ConcurrentHashMap and
 HashMaps.

 Tracing a ConcurrentHashMap$Segment to it's root goes like this (class
 name w/variable name):

 ConcurrentHashMap$Segment
 ConcurrentHashMap$Segment[]
 ConcurrentHashMap (children)
 OutInAxisOperation
 HashMap$Entry
 HashMap$Entry[]
 HashMap operationsAliasesMap
 AxisService
 HashMap$Entry (value)
 HashMap$Entry[]
 HashMap (allEndpoints)

this is the place this issue was fixed.


 AxisConfiguration (axisConfiguration)
 AxisServlet

 Hopefully that provides some insight as to where reference chain is going.

 Now, after more trial and error, I believe that I've figured out a way
 to keep it from leaking (the service has been running overnight now
 with a 32MB heap and it has not gone OOM yet):

 1. Use Axis2 1.4.1 RC1
 2. Create a pool of stubs to use so that only one thread uses a Stub at a
 time.

 #2 I tried after a lot of googling and mail archive searches where I
 found a post that indicated that the creation of and use of Stubs may
 not be thread safe. I don't know enough about the internals of axis2
 as to why this may be an issue, but hopefully it can point the
 developers in the right direction (still haven't been able to create a
 small test case).

then we can have a better look.

thanks,
Amila.



 Thanks

 -Dave

 -
 To unsubscribe, e-mail: [EMAIL PROTECTED]
 For additional commands, e-mail: [EMAIL PROTECTED]




-- 
Amila Suriarachchi,
WSO2 Inc.


Re: When do cleanup functions need to be called?

2008-07-29 Thread David Rees
On Tue, Jul 29, 2008 at 5:51 PM, David Rees [EMAIL PROTECTED] wrote:
 After pulling my hair out for the past 24 hours trying to figure out
 the root cause and desperately googling for answers, I finally found
 that calling MyStub._getService().cleanup() appears to fix the leak
 (at least the service can process requests for over an hour where it
 would die within minutes).

Unfortunately, I was mistaken. Calling cleanup() doesn't appear to
help at all, unfortunately I also switched to Axis2 1.4.1 RC1 at the
same time which appears to to have reduced the rate at which the heap
bloats, but eventually, it still suffers the same fate.

However, eventually, the service (running under Tomcat 5.5.26) still
runs out of heap space with the same symptoms - hundreds of thousands
of ConcurrentHashMap objects. These all appear to trace back to the
AxisServlet through this chain:

AxisBindingOperation - AxisBindingMessage -ConcurrentHashMap -
AxisBinding - AxisBindingOperation - AxisEndpoint - HashMap -
AxisService - OutInAxisOperation - HashMap - AxisConfiguration -
AxisServlet.

So what's going on here? I can't imagine that I'm the only one making
Axis2 client calls from a Axis2 web service and running into this
issue - either I'm doing something unique or I'm doing something
wrong...

Thanks

Dave

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Re: When do cleanup functions need to be called?

2008-07-29 Thread Amila Suriarachchi
On Wed, Jul 30, 2008 at 6:21 AM, David Rees [EMAIL PROTECTED] wrote:

 I'm developing an application using Axis2 1.4 which consists of a
 client and service. Stubs are generated using the wsdl2java tool using
 ADB.

 I am seeing what appears to be a huge resource leak when using the
 client code from the service. After processing a couple dozen service
 requests (each service requests runs a few client requests), the heap
 quickly fills up and the JVM runs out of memory.

 Analysing the heap shows hundreds of thousands of
 ConcurrentHashMap$Segment, ConcurrentHashMap$HashEntry[] and
 ReentrantLock$NonfairSync objects, and tens of thousands of HashMap,
 HashMap$Entry[], String, Object[], ArrayList,
 org.apache.axis2.description.ParameterIncludeImpl,
 org.apache.axis2.description.PolicySubject for example.

 Executing the service without making any client calls works just fine.

 Executing the client by itself in a standalone app works just fine.

 After pulling my hair out for the past 24 hours trying to figure out
 the root cause and desperately googling for answers, I finally found
 that calling MyStub._getService().cleanup() appears to fix the leak
 (at least the service can process requests for over an hour where it
 would die within minutes).

 So, my questions are:

 Is it normal to have to call ServiceClient.cleanup()?
 Why do I have to explicitly call it when using the client code from my
 service and not have to call it when running the client code as a
 standalone application?
 Should I be reusing stubs if possible?

 Code looks like this:

 public class MyService implements MyServiceSkeletonInterface {
  public MyProcessResponse myProcess() {
  MyProcessResponse res = new MyProcessResponse();
  WebServiceUtil.clientCall1();
  WebServiceUtil.clientCall2();
  res.setProcessResponseResult(OK);
  return res;
  }
 }

 public class WebServiceUtil {
  public static void clientCall1() {
  MyServiceStub stub = new MyServiceStub(http://example.com/myservice;);
  ClientCall1 req = new ClientCall1();
  ClientCall1Response res = stub.clientCall1(req);
  // Need to call this when calling using this function in a web service
  // Otherwise huge resource leak occurs
  stub._getServiceClient().cleanup();


this method should be call when the MyServiceStub (see finalize method in
ServiceClient) is garbage collected. I am not sure why it is not being
called automatically.

In RC1 we fixed a memory leak regarding this. But I can not realy think the
reason for your problem.
you can get the AxisConfiguration object like this,
AxisConfiguration axiConfiguration =
MessageContext.getCurrentMessageContext().getConfigurationContext().getAxisConfiguration();

and check whether there are some additional objects there after calling
clean up.

Please log a jira.

thanks,
Amila.



  }
 }

 Any thoughts appreciated.

 -Dave

 -
 To unsubscribe, e-mail: [EMAIL PROTECTED]
 For additional commands, e-mail: [EMAIL PROTECTED]




-- 
Amila Suriarachchi,
WSO2 Inc.