On Tue, 2015-12-15 at 16:26 +0000, Ian Jackson wrote:
> 
> +sub need_retry ($$$) {
> +    my ($jd, $dbh,$committing) = @_;
> +    return
> +     $dbh_tests->err()==7 &&
> +     ($dbh_tests->state =~ m/^40P01/); # DEADLOCK DETECTED

How come this return value doesn't need to include $committing in some way?

Also, "DEADLOCK DETECTED" presumably only occurs when fighting with some
other client, as opposed to this client being broken and doing "LOCK A;
....; LOCK A"? Or if not we have a retry limit so such code won't just spin
forever.

Ian.

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

Reply via email to