You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@phoenix.apache.org by "Rajeshbabu Chintaguntla (JIRA)" <ji...@apache.org> on 2018/02/08 05:55:01 UTC
[jira] [Comment Edited] (PHOENIX-4576) Fix LocalIndexSplitMergeIT
tests failing in master branch
[ https://issues.apache.org/jira/browse/PHOENIX-4576?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16356515#comment-16356515 ]
Rajeshbabu Chintaguntla edited comment on PHOENIX-4576 at 2/8/18 5:54 AM:
--------------------------------------------------------------------------
Currently what's happening is we are getting duplicate results because we are not skipping the child regions scanners. We are doing that properly at BaseResultIterators#getIterators on stale region exception. So as part of fix if the the exception thrown during scanner initialization throwing back to BaseResultIterators#getIterators there we handle the merge case properly.
[~jamestaylor] Please review.
In the patch I have enabled the test cases disabled at PHOENIX-4273 which are passing locally. Next time will take look at them if they failed as currently not able to get full info whats happening.
FYI [~samarthjain].
was (Author: rajeshbabu):
[~jamestaylor] Patch fixes the Local index merge failure test case. I have enabled the test cases disabled PHOENIX-4273 passing locally. Next time will take look at them if they failed as currently not able to get full info whats happening.
Please review.
> Fix LocalIndexSplitMergeIT tests failing in master branch
> ---------------------------------------------------------
>
> Key: PHOENIX-4576
> URL: https://issues.apache.org/jira/browse/PHOENIX-4576
> Project: Phoenix
> Issue Type: Bug
> Reporter: Rajeshbabu Chintaguntla
> Assignee: Rajeshbabu Chintaguntla
> Priority: Major
> Fix For: 4.14.0
>
> Attachments: PHOENIX-4576.patch
>
>
> Currenty LocalIndexSplitMergeIT#testLocalIndexScanAfterRegionsMerge is failing in master branch.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)