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

Improve recursive application of the field trimming

Rank to TopRank to BottomBulk Copy AttachmentsBulk Move AttachmentsVotersWatch issueWatchersConvert to sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Major
    • Resolution: Fixed
    • None
    • 1.31.0
    • None

    Description

      The RelFieldTrimmer has some shortcomings:

      • If the plan contains certain set ops (e.g., UNION(all=false)), even if the operator needs all fields for the correct result (CALCITE-3399), it may still make sense to apply the trimming to the children. See CALCITE-5051 for an example.
      • Same applies for a Sort with dynamic parameters in the fetch/offset, and RepeatUnions
      • The makeZeroLiteral logic in trimChildRestore(...) does not work for ARRAY / Java types.

      Attachments

        Issue Links

        Activity

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

          People

            rubenql Ruben Q L
            thomas.rebele Thomas Rebele
            Votes:
            0 Vote for this issue
            Watchers:
            6 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Time Tracking

                Estimated:
                Original Estimate - Not Specified
                Not Specified
                Remaining:
                Remaining Estimate - 0h
                0h
                Logged:
                Time Spent - 1h
                1h

                Slack

                  Issue deployment