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

Add view name to the ViewExpander

VotersWatch issueWatchersLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Major
    • Resolution: Fixed
    • None
    • 1.9.0
    • core
    • None

    Description

      The ViewExpander.expandView() call already has the schema path that contains the view but not the view name itself.
      In some context it is useful to also know the name of the view being expanded.
      current call:

      RelRoot expandView(
              RelDataType rowType,
              String queryString,
              SchemaPlus rootSchema,
              List<String> schemaPath);
      

      proposed:

      RelRoot expandView(
              RelDataType rowType,
              String queryString,
              SchemaPlus rootSchema,
              List<String> schemaPath,
              String viewName);
      

      Attachments

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            julianhyde Julian Hyde
            julienledem Julien Le Dem
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment