You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@hbase.apache.org by te...@apache.org on 2013/06/09 05:49:15 UTC

svn commit: r1491134 - /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java

Author: tedyu
Date: Sun Jun  9 03:49:15 2013
New Revision: 1491134

URL: http://svn.apache.org/r1491134
Log:
HBASE-8650 HRegionServer#updateRecoveringRegionLastFlushedSequenceId(HRegion) makes inefficient use of keySet iterator instead of entrySet iterator (Ted Yu)


Modified:
    hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java

Modified: hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java
URL: http://svn.apache.org/viewvc/hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java?rev=1491134&r1=1491133&r2=1491134&view=diff
==============================================================================
--- hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java (original)
+++ hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java Sun Jun  9 03:49:15 2013
@@ -4198,8 +4198,7 @@ public class HRegionServer implements Cl
     String previousRSName = this.getLastFailedRSFromZK(region.getEncodedName());
     Map<byte[], Long> maxSeqIdInStores = r.getMaxStoreSeqIdForLogReplay();
     long minSeqIdForLogReplay = -1;
-    for (byte[] columnFamily : maxSeqIdInStores.keySet()) {
-      Long storeSeqIdForReplay = maxSeqIdInStores.get(columnFamily);
+    for (Long storeSeqIdForReplay : maxSeqIdInStores.values()) {
       if (minSeqIdForLogReplay == -1 || storeSeqIdForReplay < minSeqIdForLogReplay) {
         minSeqIdForLogReplay = storeSeqIdForReplay;
       }