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

James Taylor commented on PHOENIX-1897:
---------------------------------------

[~sergey.b] - a VIEW over multiple tables is currently not yet supported, but 
even if it was, it would not be updatable so this would not be relevant. The 
MutationState is an internal object that tracks the state of uncommitted rows.

> Use physical table name as key in top level map for MutationState
> -----------------------------------------------------------------
>
>                 Key: PHOENIX-1897
>                 URL: https://issues.apache.org/jira/browse/PHOENIX-1897
>             Project: Phoenix
>          Issue Type: Bug
>            Reporter: James Taylor
>            Assignee: James Taylor
>
> We're currently using TableRef as the key for the top level map in 
> MutationState for uncommitted data. With the addition of transactions, there 
> are times we pass a TableRef from a SELECT statement which may have an alias. 
> This forces us to work around this for the equality checks by creating a new 
> TableRef with a null alias.
> We really should be using the physical table name as a key instead. Updates 
> to views would naturally fold into the same set of updates which is what we 
> want. Also, for indexes, we should map back to the physical table name 
> through the following logic:
> - for global indexes: get the parent table name
> - for local or shared indexes, get the physical table name and extract the 
> physical table name from the name
> - for views, get the physical table name
> - for tables, get the physical table name



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

Reply via email to