Bugs item #693861, was opened at 2003-02-26 19:43
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=376685&aid=693861&group_id=22866
Category: JBossServer
Group: v3.2
Status: Open
Resolution: None
Priority: 5
Submitted By: Bob Cotton (bcotton969)
Assigned to: Nobody/Anonymous (nobody)
Summary: PooledInvoker has a memory leak
Initial Comment:
I think I found the memory leak in the PooledInvoker.
org.jboss.invovation.pooled.server.ServerThread has an
ivar:
ObjectInputStream in;
I believe there is one ServerThread per client, and
each object read
from in.readObject() is stored in the ObjectInputStream
until that
stream is destroyed.
JProbe shows
ServerThread->
(ObjectInputStream)in->
(ArrayList)wireHandle2Object->
(Object[]) elementData->
(MarshalledInvocation)->
(Object[])args->
(MyData)
The ObjectInputStream's ArrayList has a
MarshalledInvocation for every
call that was made from the client.
I don't see any public reset method on
ObjectInputStream, so It would
seem you would need to create a new one for every
invocation inside
the while(running) loop.
- Bob
----------------------------------------------------------------------
>Comment By: Bob Cotton (bcotton969)
Date: 2003-02-28 22:22
Message:
Logged In: YES
user_id=424902
yes, 25 times slower. I'm running under JProbe now. will
keep you updated.
- Bob
----------------------------------------------------------------------
Comment By: Bill Burke (patriot1burke)
Date: 2003-02-28 21:23
Message:
Logged In: YES
user_id=176497
25 times slower!!!/?? Doesn't make sense at all. All I did was do
ObjectOutputStream.reset and sent an extra byte over the wire.
Or are you saying the PooledInvoker allowed you to be 25 times
faster? Is there a typo there?
----------------------------------------------------------------------
Comment By: Bob Cotton (bcotton969)
Date: 2003-02-28 21:13
Message:
Logged In: YES
user_id=424902
I applied your changes to Branch.3.2 and my throughput
testing went from 1250 messages/sec to 50 messages/sec.
Attached is my patch for the backport to 3.2
This is based on today's cvs, branch 3.2
----------------------------------------------------------------------
Comment By: Bill Burke (patriot1burke)
Date: 2003-02-26 20:07
Message:
Logged In: YES
user_id=176497
Thanks man. I knew there was a memory leak, I just couldn't find
it. Of course I was too lazy not to use optimizeit or something
similar
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=376685&aid=693861&group_id=22866
-------------------------------------------------------
This sf.net email is sponsored by:ThinkGeek
Welcome to geek heaven.
http://thinkgeek.com/sf
_______________________________________________
Jboss-development mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/jboss-development
