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

Field trimmer rule should use factory to create new rel nodes

        Details

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

          Description

          Currently, it either does new() or copy() to create new RelNodes.
          This will make it possible to use this rule with custom rel nodes.

          1. Text File
            OPTIQ-392.patch
            17 kB
            Ashutosh Chauhan

            Issue Links

            links to

            Web Link Pull request

              Activity

              Ascending order - Click to sort in descending order
              Hide
              Permalink
              ashutoshc Ashutosh Chauhan added a comment -

              https://github.com/apache/incubator-optiq/pull/9

              Show
              ashutoshc Ashutosh Chauhan added a comment - https://github.com/apache/incubator-optiq/pull/9
              Hide
              Permalink
              julianhyde Julian Hyde added a comment -

              Fixed in http://git-wip-us.apache.org/repos/asf/incubator-optiq/commit/d30a76bb.

              Show
              julianhyde Julian Hyde added a comment - Fixed in http://git-wip-us.apache.org/repos/asf/incubator-optiq/commit/d30a76bb .

                People

                • Assignee:
                  ashutoshc Ashutosh Chauhan
                  Reporter:
                  ashutoshc Ashutosh Chauhan
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  2 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved:

                    Development