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

Serhiy Bilousov commented on PHOENIX-1897:
------------------------------------------

bq. for views, get the physical table name

I would suspect in most cases VIEW would be base of multiple tables joined 
together.
In such case what is "physical table" ?

> 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