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

Viraj Jasani commented on PHOENIX-6058:
---------------------------------------

Based on few of my recent precommit PR results and verifying locally, I 
realized that IndexVerificationOldDesignIT is consistently failing on master 
branch after this change. 4.x is not impacted though.

 
{code:java}
java.lang.AssertionError: expected:<1> but was:<0>
        at org.junit.Assert.fail(Assert.java:89)
        at org.junit.Assert.failNotEquals(Assert.java:835)
        at org.junit.Assert.assertEquals(Assert.java:647)
        at org.junit.Assert.assertEquals(Assert.java:633)
        at 
org.apache.phoenix.end2end.IndexVerificationOldDesignIT.testIndexToolOnlyVerifyOption_viewIndex(IndexVerificationOldDesignIT.java:151)

{code}
 

 

> When maxLookback is not enabled, IndexTool should not do deep verification
> --------------------------------------------------------------------------
>
>                 Key: PHOENIX-6058
>                 URL: https://issues.apache.org/jira/browse/PHOENIX-6058
>             Project: Phoenix
>          Issue Type: Bug
>            Reporter: Kadir OZDEMIR
>            Assignee: Kadir OZDEMIR
>            Priority: Blocker
>             Fix For: 5.1.0, 4.16.0
>
>         Attachments: PHOENIX-6058.4.x.001.patch, PHOENIX-6058.4.x.002.patch, 
> PHOENIX-6058.master.001.patch
>
>
> IndexTool does two types of verification, verifying the latest row versions, 
> and verifying all row versions within the maxLookback windows. Verifying just 
> the latest version can be called shallow verification whereas verifying all 
> versions can be called deep verification. The maxLookback feature preserves 
> all row versions including delete markers within the maxLookback window. This 
> allows IndexTool to compare all versions of data rows with all versions of 
> the index rows within the window. However, due to a bug, when the maxLookback 
> feature is not enabled, i.e., the maxLookback window is set to zero, 
> IndexTool assumes the window size is infinite and thus does deep 
> verification. Since some versions of the index rows can be removed by 
> compaction independently if the maxLookback feature is not enabled, IndexTool 
> may report missing or invalid index rows by doing deep verification.  



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to