HBASE-3224 NPE in KeyValue$KVComparator.compare when compacting

git-svn-id: https://svn.apache.org/repos/asf/hbase/trunk@1034229 13f79535-47bb-0310-9956-ffa450edef68

CHANGES.txt

@@ -677,6 +677,7 @@ Release 0.90.0 - Unreleased
    HBASE-3219  Split parents are reassigned on restart and on disable/enable
    HBASE-3222  Regionserver region listing in UI is no longer ordered
    HBASE-3221  Race between splitting and disabling
+   HBASE-3224  NPE in KeyValue$KVComparator.compare when compacting
 
 
   IMPROVEMENTS

src/docbkx/book.xml

@@ -508,6 +508,15 @@ index e70ebc6..96f8c27 100644
       </section>
 
       <section xml:id="recommended_configurations"><title>Recommended Configuations</title>
+      <section xml:id="big_memory">
+        <title>Configuration for large memory machines</title>
+        <para>
+          HBase ships with a reasonable configuration that will work on nearly all
+          machine types that people might want to test with. If you have larger
+          machines you might the following configuration options helpful.
+        </para>
+
+      </section>
       <section xml:id="lzo">
       <title>LZO compression</title>
       <para>You should consider enabling LZO compression.  Its

src/main/java/org/apache/hadoop/hbase/io/HalfStoreFileReader.java

@@ -229,6 +229,10 @@ public class HalfStoreFileReader extends StoreFile.Reader {
             return 1;
           }
         }
+        if (atEnd) {
+          // skip the 'reseek' and just return 1.
+          return 1;
+        }
         return delegate.reseekTo(key, offset, length);
       }