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

Hadoop QA commented on PHOENIX-5211:
------------------------------------

{color:red}-1 overall{color}.  Here are the results of testing the latest 
attachment 
  http://issues.apache.org/jira/secure/attachment/12973119/PHOENIX-5211.patch
  against master branch at commit ae945a84dd21da8d3547419436eaba67f137dd91.
  ATTACHMENT ID: 12973119

    {color:green}+1 @author{color}.  The patch does not contain any @author 
tags.

    {color:green}+1 tests included{color}.  The patch appears to include 4 new 
or modified tests.

    {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:
    +            PTable pindexTable = PhoenixRuntime.getTable(conn, 
SchemaUtil.getTableName(schemaName, indxTable));
+    private void createAndPopulateTableAndIndexForConsistentIndex(Connection 
conn, String tableName, String indexName,
+                String fullTableName = 
SchemaUtil.getTableName(TestUtil.DEFAULT_SCHEMA_NAME, tableName);
+                String fullIndexName = 
SchemaUtil.getTableName(TestUtil.DEFAULT_SCHEMA_NAME, indexName);
+                rs = conn.createStatement().executeQuery("SELECT /*+ NO_INDEX 
*/ COUNT(*) FROM " + TABLE_NAME);
+                try (Admin admin = 
conn.unwrap(PhoenixConnection.class).getQueryServices().getAdmin();) {
+            createAndPopulateTableAndIndexForConsistentIndex(conn, 
fullTableName, fullIndexName, numRows, null);
+            rs = conn.createStatement().executeQuery("SELECT /*+ NO_INDEX */ 
COUNT(*) FROM " + TABLE_NAME);
+            // Force delete to fail (data removed but operation failed) on 
data table and check index table row remains as unverified
+            assertEquals(numRows - 1, 
getRowCount(conn.unwrap(PhoenixConnection.class).getQueryServices()

     {color:red}-1 core tests{color}.  The patch failed these unit tests:
     
./phoenix-core/target/failsafe-reports/TEST-org.apache.phoenix.end2end.DeleteIT
./phoenix-core/target/failsafe-reports/TEST-org.apache.phoenix.end2end.AlterTableIT

Test results: 
https://builds.apache.org/job/PreCommit-PHOENIX-Build/2712//testReport/
Console output: 
https://builds.apache.org/job/PreCommit-PHOENIX-Build/2712//console

This message is automatically generated.

> Consistent Immutable Global Indexes for Non-Transactional Tables
> ----------------------------------------------------------------
>
>                 Key: PHOENIX-5211
>                 URL: https://issues.apache.org/jira/browse/PHOENIX-5211
>             Project: Phoenix
>          Issue Type: Bug
>    Affects Versions: 4.13.0, 4.14.0, 5.0.0, 4.14.1
>            Reporter: Kadir OZDEMIR
>            Assignee: Gokcen Iskender
>            Priority: Major
>             Fix For: 4.15.0, 5.1.0
>
>         Attachments: PHOENIX-5211.patch
>
>          Time Spent: 11.5h
>  Remaining Estimate: 0h
>
> Without transactional tables, the immutable global indexes can get easily out 
> of sync with their data tables in Phoenix. Transactional tables require a 
> separate transaction manager, have some restrictions and performance 
> penalties. This issue is to have consistent immutable global indexes without 
> the need for using transactional tables.



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

Reply via email to