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

Create RuleOperandBuilder for building RelOptRule operands

Log workAgile BoardRank to TopRank to BottomAttach filesAttach ScreenshotBulk Copy AttachmentsBulk Move AttachmentsAdd voteVotersWatch issueWatchersCreate sub-taskConvert to sub-taskLinkCloneLabelsUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Open
    • Major
    • Resolution: Unresolved
    • 1.18.0
    • None
    • core
    • None

    Description

      Current operand / operandJ work, however it is non-trivial to pick the right one, and it does not scale as features grow.

      It would be great to have a builder there.

      He's a relevant post: https://bsideup.github.io/posts/builders_vs_java/
      TL;DR (see "Real solution'): non-fluent builders enable to write extensible and concise code.

      Some thoughts can be found in CALCITE-1166

      Attachments

        Issue Links

        is related to

        Bug - A problem which impairs or prevents the functions of the product. CALCITE-1166 Disallow sub-classes of RelOptRuleOperand

        • Major - Major loss of function.
        • Closed

        Bug - A problem which impairs or prevents the functions of the product. CALCITE-3923 Refactor how planner rules are parameterized

        • Major - Major loss of function.
        • Closed

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

          People

            Unassigned Unassigned Assign to me
            vladimirsitnikov Vladimir Sitnikov
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:

              Slack

                Issue deployment