Uploaded image for project: 'Tajo (Retired)'
  1. Tajo (Retired)
  2. TAJO-458

Visit methods of LogicalPlanVisitor should take a query block as parameter

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • None
    • 0.8.0
    • Planner/Optimizer
    • None

    Description

      A logical plan is composed of multiple query blocks. Each logical node must belong to one query block. A query block instance provides lots of information. So, it is essential information in many rewrite rules and optimizer implementations.

      However, so far, individual rewrite rule or optimizer implementation have dealt with query block directly. It may be error-prone and cause duplicated codes. This patch refactors each visitor method of LogicalPlanVisitor to take a query block as a parameter.

      I'm expecting that this change will provide more convenience for rewrite rules and optimization development.

      Attachments

        1. TAJO-458.patch
          86 kB
          Hyunsik Choi

        Activity

          People

            hyunsik Hyunsik Choi
            hyunsik Hyunsik Choi
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: