Uploaded image for project: 'Calcite'
  1. Calcite
  2. CALCITE-2560

ChainedSqlOperatorTable#add does not work

    XMLWordPrintableJSON

    Details

    • Type: Improvement
    • Status: Open
    • Priority: Major
    • Resolution: Unresolved
    • Affects Version/s: 1.17.0
    • Fix Version/s: None
    • Component/s: core
    • Labels:
      None

      Description

      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

      Julian Hyde, any thoughts on that? (I see you have updated the class to use ImmutableList)

        Attachments

          Activity

            People

            • Assignee:
              Unassigned
              Reporter:
              vladimirsitnikov Vladimir Sitnikov
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated: