Re: [Quickfix-developers] ThreadedSocketConnection: fix for memory growth under heavy load

[Oren M. <or...@qu...>]

I was testing this and a problem I noticed is that there is nothing =
calling Session::next().  Previously the thread waiting on the queue =
would timeout every second and this method would get called (via =
processStream).  This processes heartbeats and such.  With this patch, =
none of this is occuring, causing many acceptance tests to fail.

--oren
  ----- Original Message -----=20
  From: Caleb Epstein=20
  To: Oren Miller=20
  Cc: qui...@li...=20
  Sent: Wednesday, October 19, 2005 2:12 PM
  Subject: Re: [Quickfix-developers] ThreadedSocketConnection: fix for =
memory growth under heavy load


  On 10/19/05, Caleb Epstein <cal...@gm...> wrote:


    Attached is a complete patch which removes the extra thread from =
ThreadedSocketConnection as well as the functions and members which are =
no longer needed once its gone.

  Here's a slightly revised patch (one line change).  I needed to move =
the call to processStream before the recv call otherwise Initiators =
would not logon correctly.  I've now tested this code with both =
Initiators and Acceptors.


  --=20
  Caleb Epstein
  caleb dot epstein at gmail dot com