[
https://issues.apache.org/jira/browse/PHOENIX-5564?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16973761#comment-16973761
]
Hadoop QA commented on PHOENIX-5564:
------------------------------------
{color:red}-1 overall{color}. Here are the results of testing the latest
attachment
http://issues.apache.org/jira/secure/attachment/12985781/PHOENIX-5564.master.002.patch
against master branch at commit 910b72bf5d3b51a0c30ce43d9b19c0ce089cda62.
ATTACHMENT ID: 12985781
{color:green}+1 @author{color}. The patch does not contain any @author
tags.
{color:red}-1 tests included{color}. The patch doesn't appear to include
any new or modified tests.
Please justify why no new tests are needed for this
patch.
Also please list what manual steps were performed to
verify this patch.
{color:green}+1 javac{color}. The applied patch does not increase the
total number of javac compiler warnings.
{color:green}+1 release audit{color}. The applied patch does not increase
the total number of release audit warnings.
{color:red}-1 lineLengths{color}. The patch introduces the following lines
longer than 100:
+ // GlobalIndexChecker passed the index row
key. This is to build a single index row.
+ // Check if the data table row we have just
scanned matches with the index row key.
+ // If not, there is no need to build the index
row from this data table row,
+ rowCount =
GlobalIndexChecker.RebuildReturnCode.INDEX_ROW_EXISTS.getValue();
+ rowCount =
GlobalIndexChecker.RebuildReturnCode.NO_INDEX_ROW.getValue();
+ * If an index row is unverified (i.e., the row status is unverified), the
following steps are taken :
+ * (1) We generate the data row key from the index row key, and check if the
data row exists. If not, this unverified
+ * index row is skipped (i.e., not returned to the client), and it is deleted
if it is old enough. The age check is
+ * necessary in order not to delete the index rows that are currently being
updated. If the data row exists,
+ * (3) The current scanner is closed as the newly rebuilt row will not be
visible to the current scanner.
{color:green}+1 core tests{color}. The patch passed unit tests in .
Test results:
https://builds.apache.org/job/PreCommit-PHOENIX-Build/3123//testReport/
Console output:
https://builds.apache.org/job/PreCommit-PHOENIX-Build/3123//console
This message is automatically generated.
> Restructure read repair to improve readability and correctness
> --------------------------------------------------------------
>
> Key: PHOENIX-5564
> URL: https://issues.apache.org/jira/browse/PHOENIX-5564
> Project: Phoenix
> Issue Type: Improvement
> Affects Versions: 5.1.0
> Reporter: Kadir OZDEMIR
> Assignee: Kadir OZDEMIR
> Priority: Major
> Attachments: PHOENIX-5564.master.001.patch,
> PHOENIX-5564.master.002.patch
>
> Time Spent: 1h 50m
> Remaining Estimate: 0h
>
> The logic in read repair has gotten complicated and needs to be restructured
> for better readability and ensuring its correctness. More exceptions are
> needed to catch unexpected conditions and/or make sure that they do not
> happen.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
