HBASE-11846 HStore#assertBulkLoadHFileOk should log if a full HFile 
verification will be performed during a bulkload (Esteban Gutierrez)

Amending-Author: Andrew Purtell <[email protected]>


Project: http://git-wip-us.apache.org/repos/asf/hbase/repo
Commit: http://git-wip-us.apache.org/repos/asf/hbase/commit/d0b45a48
Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/d0b45a48
Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/d0b45a48

Branch: refs/heads/0.98
Commit: d0b45a488f40f082493a53fe49e3d85c6d6f0762
Parents: be81752
Author: Andrew Purtell <[email protected]>
Authored: Thu Aug 28 15:53:19 2014 -0700
Committer: Andrew Purtell <[email protected]>
Committed: Thu Aug 28 15:54:13 2014 -0700

----------------------------------------------------------------------
 .../main/java/org/apache/hadoop/hbase/regionserver/HStore.java  | 5 +++++
 1 file changed, 5 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hbase/blob/d0b45a48/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HStore.java
----------------------------------------------------------------------
diff --git 
a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HStore.java 
b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HStore.java
index 7cb5dc2..f2c86e1 100644
--- 
a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HStore.java
+++ 
b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HStore.java
@@ -642,6 +642,8 @@ public class HStore implements Store {
       }
 
       if (verifyBulkLoads) {
+        long verificationStartTime = 
EnvironmentEdgeManager.currentTimeMillis();
+        LOG.info("Full verification started for bulk load hfile: " + 
srcPath.toString());
         KeyValue prevKV = null;
         HFileScanner scanner = reader.getScanner(false, false, false);
         scanner.seekTo();
@@ -667,6 +669,9 @@ public class HStore implements Store {
           }
           prevKV = kv;
         } while (scanner.next());
+      LOG.info("Full verification complete for bulk load hfile: " + 
srcPath.toString()
+         + " took " + (EnvironmentEdgeManager.currentTimeMillis() - 
verificationStartTime)
+         + " ms");
       }
     } finally {
       if (reader != null) reader.close();

Reply via email to