continuum-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Brett Porter <br...@apache.org>
Subject Re: svn commit: r344392 - /maven/continuum/trunk/continuum-store/src/main/java/org/apache/maven/continuum/store/JdoContinuumStore.java
Date Mon, 21 Nov 2005 02:29:48 GMT
Is this stil lrequired after the other fix?

If so, can we be more specific about the caught exception?

evenisse@apache.org wrote:
> Author: evenisse
> Date: Tue Nov 15 08:48:36 2005
> New Revision: 344392
> 
> URL: http://svn.apache.org/viewcvs?rev=344392&view=rev
> Log:
> Retry the update if derby generate a lock exception
> 
> Modified:
>     maven/continuum/trunk/continuum-store/src/main/java/org/apache/maven/continuum/store/JdoContinuumStore.java
> 
> Modified: maven/continuum/trunk/continuum-store/src/main/java/org/apache/maven/continuum/store/JdoContinuumStore.java
> URL: http://svn.apache.org/viewcvs/maven/continuum/trunk/continuum-store/src/main/java/org/apache/maven/continuum/store/JdoContinuumStore.java?rev=344392&r1=344391&r2=344392&view=diff
> ==============================================================================
> --- maven/continuum/trunk/continuum-store/src/main/java/org/apache/maven/continuum/store/JdoContinuumStore.java
(original)
> +++ maven/continuum/trunk/continuum-store/src/main/java/org/apache/maven/continuum/store/JdoContinuumStore.java
Tue Nov 15 08:48:36 2005
> @@ -472,7 +472,15 @@
>                  throw new ContinuumStoreException( "Not detached: " + object );
>              }
>  
> -            pm.attachCopy( object, true );
> +            try
> +            {
> +                pm.attachCopy( object, true );
> +            }
> +            catch ( Exception e )
> +            {
> +                //We retry if we obtain an exceptio like a dead lock
> +                pm.attachCopy( object, true );
> +            }
>  
>              tx.commit();
>          }
> 
> 

Mime
View raw message