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

James Taylor commented on PHOENIX-2111:
---------------------------------------

Patch looks good. One minor nit - can you change getLinkType() to return 
LinkType instead of the serialized value (and null if not found)?
{code}
+    public static byte getLinkType(Mutation tableMutation) {
+        List<Cell> kvs = 
tableMutation.getFamilyCellMap().get(PhoenixDatabaseMetaData.TABLE_FAMILY_BYTES);
+        if (kvs != null) {
+            for (Cell kv : kvs) {
+                if (Bytes.compareTo(kv.getQualifierArray(), 
kv.getQualifierOffset(), kv.getQualifierLength(), 
PhoenixDatabaseMetaData.LINK_TYPE_BYTES, 0, 
PhoenixDatabaseMetaData.LINK_TYPE_BYTES.length) == 0) {
+                    return 
PUnsignedTinyint.INSTANCE.getCodec().decodeByte(kv.getValueArray(), 
kv.getValueOffset(), SortOrder.getDefault());
+                }
+            }
+        }
+        return (byte)-1;
+    }
{code}

> Race condition on creation of new view and adding of column to base table
> -------------------------------------------------------------------------
>
>                 Key: PHOENIX-2111
>                 URL: https://issues.apache.org/jira/browse/PHOENIX-2111
>             Project: Phoenix
>          Issue Type: Bug
>            Reporter: James Taylor
>            Assignee: Samarth Jain
>         Attachments: PHOENIX-2011-wip.patch
>
>
> There's currently a potential race condition between creating a view and 
> adding a column to the base table. The logic to find the parent row key is 
> incorrect for this scenario.



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

Reply via email to