Re: svn commit: rev 6355 - branches/svnserve-thread-pools/subversion/svnserve

Greg Hudson wrote:

>On Wed, 2003-06-25 at 19:48, Branko Čibej wrote:
>
>
>>I dunno. I'm fairly sure some generic parts could go there; for
>>instance, the request queue. Although, there's already apr_queue, which
>>I decided not to use because it has a fixed size. The thread creation
>>logic itself is fairly specific to svnserve, though, because the
>>requests aren't independent (one thread can block until another request
>>is processed -- some diff and merge cases and externals handling are
>>examples).
>>
>>
>
>When you say the logic is specific to svnserve, you mean that svnserve
>requires a policy which not every server needs, right? (Specifically, a
>connection never blocks waiting for a thread to become idle; if no idle
>thread is available, a new one is always created.)
>
Yes, that's right.

>That seems like something you could parameterize in init_threads()
>without adding too much complexity. Though I can see how it would be
>extra work.
>
>
I expect so. But I think it would be worth the trouble, if the APR
people agreed that this is something useful to have in apr-util.

-- 
Brane Čibej   <brane_at_xbc.nu>   http://www.xbc.nu/brane/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org