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

Hadoop QA commented on PHOENIX-5550:
------------------------------------

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

    {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:green}+1 lineLengths{color}.  The patch does not introduce lines 
longer than 100

    {color:green}+1 core tests{color}.  The patch passed unit tests in .

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

This message is automatically generated.

> Scan after local index creation on table having data giving wrong results 
> when long view index id disabled
> ----------------------------------------------------------------------------------------------------------
>
>                 Key: PHOENIX-5550
>                 URL: https://issues.apache.org/jira/browse/PHOENIX-5550
>             Project: Phoenix
>          Issue Type: Bug
>            Reporter: Rajeshbabu Chintaguntla
>            Assignee: Rajeshbabu Chintaguntla
>            Priority: Blocker
>             Fix For: 4.15.0, 5.1.0
>
>         Attachments: PHOENIX-5550.patch
>
>
> After PHOENIX-3547 and PHOENIX-5104 querying after local index creation on 
> the table having data is failing because of the view index id type mismatches.
> The client is sending the SMALLINT as view index id data type.
> {noformat}
>     private PDataType getViewIndexDataType() throws SQLException {
>         boolean supportsLongViewIndexId = 
> connection.getQueryServices().getProps().getBoolean(
>                                 QueryServices.LONG_VIEW_INDEX_ENABLED_ATTRIB,
>                                 
> QueryServicesOptions.DEFAULT_LONG_VIEW_INDEX_ENABLED);
>         return supportsLongViewIndexId ? 
> MetaDataUtil.getViewIndexIdDataType() : 
> MetaDataUtil.getLegacyViewIndexIdDataType();
>     }
> {noformat}
> But in the  create index api in endpoint impl is considering LONG as view 
> index id data type.
> {noformat}
>                         PDataType dataType = 
> MetaDataUtil.getViewIndexIdDataType();
>                         Object val = dataType.toObject(seqValue, 
> PLong.INSTANCE);
>                         byte[] bytes = new byte[dataType.getByteSize() + 1];
>                         dataType.toBytes(val, bytes, 0);
>                         Cell indexIdCell =
>                                 
> PhoenixKeyValueUtil.newKeyValue(cell.getRowArray(),
>                                     cell.getRowOffset(), cell.getRowLength(),
>                                     cell.getFamilyArray(), 
> cell.getFamilyOffset(),
>                                     cell.getFamilyLength(), 
> VIEW_INDEX_ID_BYTES, 0,
>                                     VIEW_INDEX_ID_BYTES.length, 
> cell.getTimestamp(), bytes, 0,
>                                     bytes.length, cell.getType());
> {noformat}
> {noformat}
>                 if (indexId != null) {
>                     builder.setViewIndexId(indexId);
>                     builder.setViewIndexIdType(PLong.INSTANCE.getSqlType());
>                 }
> {noformat}
>  
> Most of the local index tests failing If we set false value to the following 
> configuration.
>   <property>
>     <!--
>     We have some hardcoded viewIndex ids in the IT tests which assumes 
> viewIndexId is of type Long.
>     However the default viewIndexId type is set to "short" by default until 
> we upgrade all clients to support
>     long viewIndex ids.
>     -->
>     <name>phoenix.index.longViewIndex.enabled</name>
>     <value>false</value>
>   </property>



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

Reply via email to