You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by je...@apache.org on 2013/08/18 08:14:01 UTC

svn commit: r1515075 - /hbase/trunk/hbase-client/src/main/java/org/apache/hadoop/hbase/client/AsyncProcess.java

Author: jeffreyz
Date: Sun Aug 18 06:14:01 2013
New Revision: 1515075

URL: http://svn.apache.org/r1515075
Log:
HBASE-9257: TestAsyncProcess#testFailAndSuccess fails sometime due to a race condition

Modified:
    hbase/trunk/hbase-client/src/main/java/org/apache/hadoop/hbase/client/AsyncProcess.java

Modified: hbase/trunk/hbase-client/src/main/java/org/apache/hadoop/hbase/client/AsyncProcess.java
URL: http://svn.apache.org/viewvc/hbase/trunk/hbase-client/src/main/java/org/apache/hadoop/hbase/client/AsyncProcess.java?rev=1515075&r1=1515074&r2=1515075&view=diff
==============================================================================
--- hbase/trunk/hbase-client/src/main/java/org/apache/hadoop/hbase/client/AsyncProcess.java (original)
+++ hbase/trunk/hbase-client/src/main/java/org/apache/hadoop/hbase/client/AsyncProcess.java Sun Aug 18 06:14:01 2013
@@ -498,8 +498,8 @@ class AsyncProcess<CResult> {
       if (callback != null) {
         callback.failure(originalIndex, region, row, throwable);
       }
-      this.hasError.set(true);
       errors.add(throwable, row, location);
+      this.hasError.set(true);
     }
 
     return canRetry;