hbase-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "jiraposter@reviews.apache.org (Commented) (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HBASE-4585) Avoid seek operation when current kv is deleted
Date Fri, 14 Oct 2011 05:30:13 GMT

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

jiraposter@reviews.apache.org commented on HBASE-4585:
------------------------------------------------------


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/2384/#review2579
-----------------------------------------------------------


+1. nice fix

- ramkrishna


On 2011-10-14 04:15:21, Liyin wrote:
bq.  
bq.  -----------------------------------------------------------
bq.  This is an automatically generated e-mail. To reply, visit:
bq.  https://reviews.apache.org/r/2384/
bq.  -----------------------------------------------------------
bq.  
bq.  (Updated 2011-10-14 04:15:21)
bq.  
bq.  
bq.  Review request for hbase, Dhruba Borthakur, Michael Stack, Jonathan Gray, Mikhail Bautin,
Kannan Muthukkaruppan, Jerry Chen, Karthik Ranganathan, and Nicolas Spiegelberg.
bq.  
bq.  
bq.  Summary
bq.  -------
bq.  
bq.  When the current kv is deleted during the matching in the ScanQueryMatcher, currently
the matcher will return skip and continue to seek.
bq.  Actually, if the current kv is deleted because of family deleted or column deleted, the
matcher should seek to next col.
bq.  If the current kv is deleted because of version deleted, the matcher should just return
skip.
bq.  
bq.  The patch for 89-fb and apache trunk should be the same. 
bq.  I will attach different patches rebase on 89-fb and apache trunk in the jira.
bq.  
bq.  
bq.  This addresses bug HBASE-4585.
bq.      https://issues.apache.org/jira/browse/HBASE-4585
bq.  
bq.  
bq.  Diffs
bq.  -----
bq.  
bq.    src/test/java/org/apache/hadoop/hbase/regionserver/TestScanDeleteTracker.java afb3fcc

bq.    src/test/java/org/apache/hadoop/hbase/regionserver/TestBlocksRead.java e8ac027 
bq.    src/main/java/org/apache/hadoop/hbase/regionserver/ScanQueryMatcher.java a1d7de5 
bq.    src/main/java/org/apache/hadoop/hbase/regionserver/DeleteTracker.java b425bf2 
bq.    src/main/java/org/apache/hadoop/hbase/regionserver/ScanDeleteTracker.java 1a891a6 
bq.  
bq.  Diff: https://reviews.apache.org/r/2384/diff
bq.  
bq.  
bq.  Testing
bq.  -------
bq.  
bq.  Running all the unit tests.
bq.  
bq.  
bq.  Thanks,
bq.  
bq.  Liyin
bq.  
bq.


                
> Avoid seek operation when current kv is deleted
> -----------------------------------------------
>
>                 Key: HBASE-4585
>                 URL: https://issues.apache.org/jira/browse/HBASE-4585
>             Project: HBase
>          Issue Type: Improvement
>            Reporter: Liyin Tang
>            Assignee: Liyin Tang
>         Attachments: hbase-4585-89.patch, hbase-4585-trunk.patch
>
>
> When the current kv is deleted during the matching in the ScanQueryMatcher, currently
the matcher will return skip and continue to seek.
> Actually, if the current kv is deleted because of family deleted or column deleted, the
matcher should seek to next col.
> If the current kv is deleted because of version deleted, the matcher should just return
skip.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Mime
View raw message