Return-Path: Delivered-To: apmail-hbase-commits-archive@www.apache.org Received: (qmail 3255 invoked from network); 5 Apr 2011 04:52:12 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.3) by minotaur.apache.org with SMTP; 5 Apr 2011 04:52:12 -0000 Received: (qmail 31454 invoked by uid 500); 5 Apr 2011 04:52:11 -0000 Delivered-To: apmail-hbase-commits-archive@hbase.apache.org Received: (qmail 31360 invoked by uid 500); 5 Apr 2011 04:52:10 -0000 Mailing-List: contact commits-help@hbase.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@hbase.apache.org Delivered-To: mailing list commits@hbase.apache.org Received: (qmail 31349 invoked by uid 99); 5 Apr 2011 04:52:09 -0000 Received: from nike.apache.org (HELO nike.apache.org) (192.87.106.230) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 05 Apr 2011 04:52:09 +0000 X-ASF-Spam-Status: No, hits=-2000.0 required=5.0 tests=ALL_TRUSTED X-Spam-Check-By: apache.org Received: from [140.211.11.4] (HELO eris.apache.org) (140.211.11.4) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 05 Apr 2011 04:52:07 +0000 Received: by eris.apache.org (Postfix, from userid 65534) id 644F12388901; Tue, 5 Apr 2011 04:51:46 +0000 (UTC) Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit 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 -0000 To: commits@hbase.apache.org From: jdcryans@apache.org X-Mailer: svnmailer-1.0.8 Message-Id: <20110405045146.644F12388901@eris.apache.org> X-Virus-Checked: Checked by ClamAV on apache.org 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 regionsBySize = server.getCopyOfOnlineRegionsSortedBySize(); - Set excludedRegions = new TreeSet(); + Set excludedRegions = new HashSet(); boolean flushedOne = false; while (!flushedOne) {