hbase-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "HBase Review Board (JIRA)" <j...@apache.org>
Subject [jira] Commented: (HBASE-3082) For ICV gets, first look in MemStore before reading StoreFiles
Date Thu, 21 Oct 2010 04:32:16 GMT

    [ https://issues.apache.org/jira/browse/HBASE-3082?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12923314#action_12923314
] 

HBase Review Board commented on HBASE-3082:
-------------------------------------------

Message from: thekhemani@facebook.com

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://review.cloudera.org/r/1053/
-----------------------------------------------------------

(Updated 2010-10-20 21:31:17.831889)


Review request for hbase and Jonathan Gray.


Changes
-------

I think I have taken care of all the review feedback


Summary
-------

Ensure that during incrementColumnValue() the scan triggered by the get() does an early exit
if it finds the KV in the memstore.


This addresses bug HBASE-3082.
    http://issues.apache.org/jira/browse/HBASE-3082


Diffs (updated)
-----

  src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java 0e3940c 
  src/main/java/org/apache/hadoop/hbase/regionserver/InternalScan.java PRE-CREATION 
  src/main/java/org/apache/hadoop/hbase/regionserver/StoreScanner.java 4775fc8 

Diff: http://review.cloudera.org/r/1053/diff


Testing
-------

I have been testing it on my cluster. No unit testing yet.


Thanks,

khemani




> For ICV gets, first look in MemStore before reading StoreFiles
> --------------------------------------------------------------
>
>                 Key: HBASE-3082
>                 URL: https://issues.apache.org/jira/browse/HBASE-3082
>             Project: HBase
>          Issue Type: Improvement
>          Components: regionserver
>            Reporter: Jonathan Gray
>            Assignee: Prakash Khemani
>
> For incrementColumnValue operations, it is possible to check MemStore for the column
being incremented without sacrificing correctness.  If the column is not found in MemStore,
we would then have to do a normal Get that opens/checks all StoreFiles for the given Store.
> In practice, this makes increment operations significantly faster for recently/frequently
incremented columns.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


Mime
View raw message