[ 
https://issues.apache.org/jira/browse/CALCITE-1028?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Maryann Xue updated CALCITE-1028:
---------------------------------
    Attachment: CALCITE-1028.patch

Could you please help review this patch, [~julianhyde]? It also includes a bug 
fix for populateMaterializations():
populateMaterialization() should use the same "viewSchemaPath" as used in 
defineMaterialization() for parsing and validating the sql. Otherwise if 
"viewSchemaPath" happens to be different from the materialization schema path 
itself, validation would fail in populateMaterialization().

> Move populate materializations after sql-to-rel conversion
> ----------------------------------------------------------
>
>                 Key: CALCITE-1028
>                 URL: https://issues.apache.org/jira/browse/CALCITE-1028
>             Project: Calcite
>          Issue Type: Improvement
>            Reporter: Maryann Xue
>            Assignee: Maryann Xue
>            Priority: Minor
>              Labels: phoenix
>             Fix For: 1.6.0
>
>         Attachments: CALCITE-1028.patch
>
>
> This is to allow Phoenix to figure out which materializations are concerned 
> with the query and thus have a chance to define them right before 
> materializations are populated.



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

Reply via email to