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

James Taylor commented on PHOENIX-1297:
---------------------------------------

What I was thinking for the PDataType change was along these lines:
- Add and use a new constructor for all array types that just takes the typeId 
as an argument and delegates to the other one like this:
{code}
    private PDataType(int baseTypeId) {
        this(fromTypeId(baseTypeId).getSqlTypeName() + " " + ARRAY_TYPE_SUFFIX,
             PDataType.ARRAY_TYPE_BASE + fromTypeId(baseTypeId).getSqlType(),
             PhoenixArray.class,
             null);
    }
    
    public static final String ARRAY_TYPE_SUFFIX = "ARRAY";
{code}
- Use the ARRAY_TYPE_SUFFIX in your code to append to the type to get the full 
type name (with a space char in between)

> Adding utility methods to get primary key information from the optimized 
> query plan
> -----------------------------------------------------------------------------------
>
>                 Key: PHOENIX-1297
>                 URL: https://issues.apache.org/jira/browse/PHOENIX-1297
>             Project: Phoenix
>          Issue Type: Task
>    Affects Versions: 5.0.0, 4.2, 3.2
>            Reporter: Samarth Jain
>            Assignee: Samarth Jain
>         Attachments: PHOENIX-1297.patch, PHOENIX-1297_v2.patch, 
> PHOENIX-1297_v3.patch, PHOENIX-1297_v4.patch
>
>




--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to