[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: svn commit: rev 6161 - trunk/subversion/libsvn_fs

From: Branko Čibej <brane_at_xbc.nu>
Date: 2003-06-06 03:58:13 CEST

joe@tigris.org wrote:

>Author: joe
>Date: Thu Jun 5 16:54:46 2003
>New Revision: 6161
> trunk/subversion/libsvn_fs/trail.c
>Fix for occasional unhandled deadlock errors, as seen, for example,
>in: http://www.contactor.se/~dast/svn/archive-2003-03/0152.shtml
>* subversion/libsvn_fs/trail.c (svn_fs__retry_txn): Check the whole
>stack for a deadlock error.
I wonder, shouldn't we just try to figure out where we're wrapping the
deadlock errors? I mean, it doesn't make any sense to wrap them if we
know that a deadlock must always result in a retry. It seems to me that
this change is just hiding existing error handling bugs.

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
Received on Fri Jun 6 03:56:53 2003

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.