You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@hbase.apache.org by "Andrey Elenskiy (JIRA)" <ji...@apache.org> on 2018/12/28 00:54:00 UTC

[jira] [Created] (HBASE-21653) NewVersionBehaviorTracker.checkVersions() should allow cell type to be DELETE

Andrey Elenskiy created HBASE-21653:
---------------------------------------

             Summary: NewVersionBehaviorTracker.checkVersions() should allow cell type to be DELETE
                 Key: HBASE-21653
                 URL: https://issues.apache.org/jira/browse/HBASE-21653
             Project: HBase
          Issue Type: Bug
          Components: API
    Affects Versions: 2.1.1, 2.0.0
            Reporter: Andrey Elenskiy


`MajorCompactionScanQueryMatcher.match()` states that  "7. Delete marker need to be version counted together with puts they affect" which corresponds to a code path that can happen when KEEP_DELETED_CELLS is true. However, `NewVersionBehaviorTracker.checkVersions()` asserts that type cannot be DELETE.

The AssertionError can be verified by running `TestKeepDeletes.testBasicScenario` with "-Dhbase.tests.new.version.behavior=true" after fixing it up to work as expected with NEW_VERSIONS_BEHAVIOR:
{{--- a/hbase-server/src/test/java/org/apache/hadoop/hbase/regionserver/TestKeepDeletes.java}}
{{+++ b/hbase-server/src/test/java/org/apache/hadoop/hbase/regionserver/TestKeepDeletes.java}}
{{@@ -135,7 +135,7 @@ public class TestKeepDeletes {}}
{{     g.setMaxVersions();}}
{{     g.setTimeRange(0L, ts+2);}}
{{     Result r = region.get(g);}}
{{-    checkResult(r, c0, c0, T2, T1);}}
{{+    checkResult(r, c0, c0, T2);}}

{{     // flush}}
{{     region.flush(true);}}

 

Some more info in the following comment: https://issues.apache.org/jira/browse/HBASE-21545?focusedCommentId=16719510&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#comment-16719510



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)