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

Add RelBuilder field() method to reference aliased relations not on top of stack

        Details

        • Type: Improvement
        • Status: Closed
        • Priority: Minor
        • Resolution: Fixed
        • Affects Version/s: None
        • Fix Version/s: 1.11.0
        • Component/s: core
        • Labels:
          None

          Description

          It would be convenient to reference aliased tables that are not on top of the RelBuilder stack.

            Issue Links

            links to

            Web Link PR #332

              Activity

              Ascending order - Click to sort in descending order
              Hide
              Permalink
              jbalint@gmail.com Jess Balint added a comment -

              I don't see anything preventing conflicting alias names. should this be validated in the Frame constructor?

              Show
              jbalint@gmail.com Jess Balint added a comment - I don't see anything preventing conflicting alias names. should this be validated in the Frame constructor?
              Hide
              Permalink
              julianhyde Julian Hyde added a comment -

              Yes, I think that would be useful. But if we allow null alias names (I don't remember) we should continue to allow that.

              Show
              julianhyde Julian Hyde added a comment - Yes, I think that would be useful. But if we allow null alias names (I don't remember) we should continue to allow that.
              Hide
              Permalink
              julianhyde Julian Hyde added a comment -

              Fixed in http://git-wip-us.apache.org/repos/asf/calcite/commit/ffe1a68d. Thanks for the PR, Jess Balint!

              Show
              julianhyde Julian Hyde added a comment - Fixed in http://git-wip-us.apache.org/repos/asf/calcite/commit/ffe1a68d . Thanks for the PR, Jess Balint !
              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:
                  jbalint@gmail.com Jess Balint
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  2 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved:

                    Development