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

Create a Shuttle for visiting linq4j expressions without mutating them

        Details

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

          Description

          Create a shuttle for visiting linq4j expressions without mutating them. Rename the current Visitor class to Shuttle, create a new Visitor<R> interface, and a default implementation VisitorImpl<R>, and add a method <R> R Node.accept(Visitor<R> visitor).

          Note that accept's return type is R: not necessarily a node, in fact often Void.

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            julianhyde Julian Hyde added a comment -

            Fixed in http://git-wip-us.apache.org/repos/asf/calcite/commit/267e2f3e.

            Show
            julianhyde Julian Hyde added a comment - Fixed in http://git-wip-us.apache.org/repos/asf/calcite/commit/267e2f3e .
            Hide
            Permalink
            julianhyde Julian Hyde added a comment -

            Resolved in release 1.11.0 (2017-01-11).

            Show
            julianhyde Julian Hyde added a comment - Resolved in release 1.11.0 (2017-01-11).

              People

              • Assignee:
                julianhyde Julian Hyde
                Reporter:
                julianhyde Julian Hyde
              • Votes:
                0 Vote for this issue
                Watchers:
                1 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Development