Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: None
    • Labels:

      Description

      Convert `SqlNode` from a class to an interface.

      This would allow:

      1. Make `SqlNodeList` extend `AbstractList` or `ArrayList` rather than wrapping a list.
      2. Make `SqlSelect`, `SqlWindow` and similar classes implement `SqlNode` directly rather than extending `SqlCall`. Their arguments would be stored in named, typed fields rather than the operand array. They could still have a `getOperator()` method (but not `setOperator`)
      3. Maybe convert `SqlCall.operands` array to an immutable list.
      4. Remove `implements Cloneable`. There are better ways.

      ---------------- Imported from GitHub ----------------
      Url: https://github.com/julianhyde/optiq/issues/147
      Created by: julianhyde
      Labels: cleanup,
      Created at: Thu Feb 20 22:16:24 CET 2014
      State: closed

        Attachments

          Activity

            People

            • Assignee:
              Unassigned
              Reporter:
              github-import GitHub Import
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: