You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@calcite.apache.org by "Maryann Xue (JIRA)" <ji...@apache.org> on 2017/01/12 00:54:16 UTC

[jira] [Resolved] (CALCITE-1571) Could not resolve VIEW with SimpleCalciteSchema

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

Maryann Xue resolved CALCITE-1571.
----------------------------------
       Resolution: Fixed
    Fix Version/s: 1.12.0

Thank you, [~julianhyde], for the review! Fixed in https://git1-us-west.apache.org/repos/asf?p=calcite.git;a=commit;h=2c2b883.

> Could not resolve VIEW with SimpleCalciteSchema
> -----------------------------------------------
>
>                 Key: CALCITE-1571
>                 URL: https://issues.apache.org/jira/browse/CALCITE-1571
>             Project: Calcite
>          Issue Type: Bug
>          Components: core
>    Affects Versions: 1.11.0
>            Reporter: Maryann Xue
>            Assignee: Maryann Xue
>             Fix For: 1.12.0
>
>
> The current implementation (ever since its first version) of SimpleCalciteSchema.getImplicitTableBasedOnNullaryFunction() is wrong, but  it was not revealed until [~julianhyde]'s check-in for CALCITE-1563.
> The implementation 1) did not use "tableName" in finding the functions at all; 2) was caching based.
> {code}
>   protected TableEntry getImplicitTableBasedOnNullaryFunction(String tableName,
>       boolean caseSensitive) {
>     for (String s : schema.getFunctionNames()) {
>       for (Function function : schema.getFunctions(s)) {
>         if (function instanceof TableMacro
>             && function.getParameters().isEmpty()) {
>           final Table table = ((TableMacro) function).apply(ImmutableList.of());
>           return tableEntry(tableName, table);
>         }
>       }
>     }
>     return null;
>   }
> {code}



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