You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@calcite.apache.org by "Vladimir Sitnikov (JIRA)" <ji...@apache.org> on 2018/09/16 13:42:00 UTC

[jira] [Created] (CALCITE-2560) ChainedSqlOperatorTable#add does not work

Vladimir Sitnikov created CALCITE-2560:
------------------------------------------

             Summary: ChainedSqlOperatorTable#add does not work
                 Key: CALCITE-2560
                 URL: https://issues.apache.org/jira/browse/CALCITE-2560
             Project: Calcite
          Issue Type: Improvement
          Components: core
    Affects Versions: 1.17.0
            Reporter: Vladimir Sitnikov
            Assignee: Julian Hyde


{{tableList}} is {{ImmutableList}}, so {{add}} method won't work: https://github.com/apache/calcite/blob/74803d0ac093a291b38f629fdc3dc720bcc5cd4c/core/src/main/java/org/apache/calcite/sql/util/ChainedSqlOperatorTable.java#L64

We should either remove {{add} method, or avoid use of {{ImmutableList}} int that case.

A bit of fun could be to use persistent collections instead of simple immutable ones :)

[~julianhyde], any thoughts on that? (I see you have updated the class to use ImmutableList)



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)