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

SqlToRelConverter mixes uses of configured RelBuilder and Logical.create methods.

Attach filesAttach ScreenshotVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Bug
    • Status: Resolved
    • Major
    • Resolution: Duplicate
    • None
    • None
    • None
    • None

    Description

      SqlToRelConverter takes a configuration which allows setting a RelBuilderFactory, which it does use in some places, but in others it directly calls LogicalWhatever.create to build a node.

      All RelNode builder calls should (as much as possible) build nodes using the configured RelBuilder.

      Attachments

        Issue Links

        duplicates

        Bug - A problem which impairs or prevents the functions of the product. CALCITE-4262 In SqlToRelConverter, use RelBuilder for creating all relational expressions

        • Major - Major loss of function.
        • In Progress

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

          People

            Unassigned Unassigned
            ian.bertolacci Ian Bertolacci
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment