You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by st...@apache.org on 2011/10/20 00:05:38 UTC

svn commit: r1186531 - in /hbase/trunk: CHANGES.txt src/test/java/org/apache/hadoop/hbase/avro/TestAvroServer.java

Author: stack
Date: Wed Oct 19 22:05:37 2011
New Revision: 1186531

URL: http://svn.apache.org/viewvc?rev=1186531&view=rev
Log:
HBASE-4621 TestAvroServer fails quite often intermittently

Modified:
    hbase/trunk/CHANGES.txt
    hbase/trunk/src/test/java/org/apache/hadoop/hbase/avro/TestAvroServer.java

Modified: hbase/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/hbase/trunk/CHANGES.txt?rev=1186531&r1=1186530&r2=1186531&view=diff
==============================================================================
--- hbase/trunk/CHANGES.txt (original)
+++ hbase/trunk/CHANGES.txt Wed Oct 19 22:05:37 2011
@@ -370,6 +370,7 @@ Release 0.92.0 - Unreleased
                spammed when too many store files
    HBASE-4620  I broke the build when I submitted HBASE-3581 (Send length
                of the rpc response)
+   HBASE-4621  TestAvroServer fails quite often intermittently (Akash Ashok)
 
   TESTS
    HBASE-4450  test for number of blocks read: to serve as baseline for expected

Modified: hbase/trunk/src/test/java/org/apache/hadoop/hbase/avro/TestAvroServer.java
URL: http://svn.apache.org/viewvc/hbase/trunk/src/test/java/org/apache/hadoop/hbase/avro/TestAvroServer.java?rev=1186531&r1=1186530&r2=1186531&view=diff
==============================================================================
--- hbase/trunk/src/test/java/org/apache/hadoop/hbase/avro/TestAvroServer.java (original)
+++ hbase/trunk/src/test/java/org/apache/hadoop/hbase/avro/TestAvroServer.java Wed Oct 19 22:05:37 2011
@@ -122,10 +122,12 @@ public class TestAvroServer {
     impl.disableTable(tableAname);
     assertFalse(impl.isTableEnabled(tableAname));
 
+    long oldMaxFileSize = impl.describeTable(tableAname).maxFileSize;
     tableA.maxFileSize = 123456L;
     impl.modifyTable(tableAname, tableA);
+  
     // It can take a while for the change to take effect.  Wait here a while.
-    while(impl.describeTable(tableAname) == null) {
+    while(impl.describeTable(tableAname).maxFileSize == oldMaxFileSize) {
       Threads.sleep(100);
     }