[Quickfix-developers] Re: Session::send sync issue

[<OM...@th...>]

I can't believe I missed this suggestion before.  Someone actually ran into
this problem today.  I diagnosed quickly enough but should have made this
change when you sent it 2 months ago!!!  I happened to run into this
because I was browsing your emails since you always have good suggestions
concerning threading issues.  Needless to say I'm checking this in today.

--oren



                                                                                                                                       
                      Gene Gorokhovsky                                                                                                 
                      <musor102@yahoo.c        To:       OM...@th..., qui...@li...           
                      om>                      cc:                                                                                     
                                               Subject:  Session::send sync issue                                                      
                      11/05/2002 12:03                                                                                                 
                      AM                                                                                                               
                                                                                                                                       
                                                                                                                                       




Locker(m_mutex) should be moved from Session::send
to Session::sendRaw, otherwise there is a race
condition between app-level and admin-level messages
if the application spawns separate (from the primary
Session queue processing) threads to send app-level
messages. I happened to actually run into that because
I have another thread per session that provides async
sendToTarget outgoing queue.

Gene

__________________________________________________
Do you Yahoo!?
HotJobs - Search new jobs daily now
http://hotjobs.yahoo.com/