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

Geoffrey Jacoby commented on PHOENIX-5710:
------------------------------------------

[~kozdemir] Please add a test that fails without the change and passes with it. 
Also an explanatory comment would be good. 

> dataEmptyKeyValueRef is not set correctly in IndexMaintainer
> ------------------------------------------------------------
>
>                 Key: PHOENIX-5710
>                 URL: https://issues.apache.org/jira/browse/PHOENIX-5710
>             Project: Phoenix
>          Issue Type: Improvement
>    Affects Versions: 5.0.0, 4.14.3
>            Reporter: Kadir OZDEMIR
>            Assignee: Kadir OZDEMIR
>            Priority: Major
>         Attachments: PHOENIX-5710.master.001.patch
>
>
> I noticed that the following line in IndexMaintainer.initCachedState() :
> dataEmptyKeyValueRef = new 
> ColumnReference(emptyKeyValueCFPtr.copyBytesIfNecessary(), emptyKvQualifier);
> This is incorrect. It should have been the following:
> dataEmptyKeyValueRef = new ColumnReference(dataEmptyKeyValueCF, 
> emptyKvQualifier);



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

Reply via email to