hbase-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From jdcry...@apache.org
Subject svn commit: r1088894 - in /hbase/trunk: CHANGES.txt src/main/java/org/apache/hadoop/hbase/regionserver/MemStoreFlusher.java
Date Tue, 05 Apr 2011 04:51:46 GMT
Author: jdcryans
Date: Tue Apr  5 04:51:46 2011
New Revision: 1088894

URL: http://svn.apache.org/viewvc?rev=1088894&view=rev
Log:
HBASE-3733  MemStoreFlusher.flushOneForGlobalPressure() shouldn't
            be using TreeSet for HRegion (Ted Yu via J-D)

Modified:
    hbase/trunk/CHANGES.txt
    hbase/trunk/src/main/java/org/apache/hadoop/hbase/regionserver/MemStoreFlusher.java

Modified: hbase/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/hbase/trunk/CHANGES.txt?rev=1088894&r1=1088893&r2=1088894&view=diff
==============================================================================
--- hbase/trunk/CHANGES.txt (original)
+++ hbase/trunk/CHANGES.txt Tue Apr  5 04:51:46 2011
@@ -61,6 +61,8 @@ Release 0.91.0 - Unreleased
    HBASE-3238  HBase needs to have the CREATE permission on the parent of its
                ZooKeeper parent znode (Alex Newman via Stack)
    HBASE-3728  NPE in HTablePool.closeTablePool (Ted Yu via Stack)
+   HBASE-3733  MemStoreFlusher.flushOneForGlobalPressure() shouldn't
+               be using TreeSet for HRegion (Ted Yu via J-D)
 
   IMPROVEMENTS
    HBASE-3290  Max Compaction Size (Nicolas Spiegelberg via Stack)  

Modified: hbase/trunk/src/main/java/org/apache/hadoop/hbase/regionserver/MemStoreFlusher.java
URL: http://svn.apache.org/viewvc/hbase/trunk/src/main/java/org/apache/hadoop/hbase/regionserver/MemStoreFlusher.java?rev=1088894&r1=1088893&r2=1088894&view=diff
==============================================================================
--- hbase/trunk/src/main/java/org/apache/hadoop/hbase/regionserver/MemStoreFlusher.java (original)
+++ hbase/trunk/src/main/java/org/apache/hadoop/hbase/regionserver/MemStoreFlusher.java Tue
Apr  5 04:51:46 2011
@@ -23,10 +23,10 @@ import java.io.IOException;
 import java.lang.management.ManagementFactory;
 import java.util.ConcurrentModificationException;
 import java.util.HashMap;
+import java.util.HashSet;
 import java.util.Map;
 import java.util.Set;
 import java.util.SortedMap;
-import java.util.TreeSet;
 import java.util.concurrent.BlockingQueue;
 import java.util.concurrent.DelayQueue;
 import java.util.concurrent.Delayed;
@@ -151,7 +151,7 @@ class MemStoreFlusher extends Thread imp
     SortedMap<Long, HRegion> regionsBySize =
         server.getCopyOfOnlineRegionsSortedBySize();
 
-    Set<HRegion> excludedRegions = new TreeSet<HRegion>();
+    Set<HRegion> excludedRegions = new HashSet<HRegion>();
 
     boolean flushedOne = false;
     while (!flushedOne) {



Mime
View raw message