Re: [patch] RE: immediate-sendsms-reply
Alexander Malysh <[email protected]>
| Newsgroups | gmane.comp.mobile.kannel.devel |
|---|---|
| Message-ID | <[email protected]> |
Hi, yes, it could explain why we have user reports with too many file descriptors while sending through sendsms interface. Do you have any idea/patch how to fix it? Thanks, Alex Dziugas Baltrunas schrieb: > Hi, > > I must confirm that we _need_ to commit the proposed patch either in > current or modified behaviour since the bug still remains in current > CVS head. > > On 3/22/05, Kalle Marjola <[email protected]> wrote: >> On Mon, 2005-03-21 at 13:11, Alexander Malysh wrote: >>> Hi, >>> >>> Kalle Marjola wrote: >>> >>>> Ok, I though over this and a bit updated your patch so that it only >>>> waits if there are 'partially handled HTTP request' in the air (i.e. has >>>> already sent the message onward to bearerbox but has not yet stored the >>>> http client into disctionary). Moreover, the processing time between >>>> sending the message to bb and saving the client into dictionary has been >>>> minimized.. >>> why not first put http client into dictionary and then send msg to >>> bearerbox? >>> >> I thought about that and did not do that way because that would overload >> several functions with a list of new parameters, >> >> >> -- >> &Kalle Marjola ::: Development ::: Helsinki ::: Enpocket >> >> >> > > > -- > Dziugas > >