Explain::explainPipeline() should accept PlanExecutorPipeline* instead of PlanExecutor* as a parameter

XMLWordPrintableJSON

    • Type: Improvement
    • Resolution: Fixed
    • Priority: Minor - P4
    • 8.2.0-rc0
    • Affects Version/s: None
    • Component/s: None
    • Query Execution
    • Minor Change
    • QE 2025-06-23, QE 2025-07-07, QE 2025-07-21
    • None
    • 3
    • TBD
    • None
    • None
    • None
    • None
    • None
    • None
    • None

      PlanExecutor* exec is immediately dynamically casted to PlanExecutorPipeline* and checked to be non-nullptr anyway. In the places where this method is called the exec is already checked to be PlanExecutorPipeline, by changing the signature we could perform one less downcast.

              Assignee:
              Vlad Dragutoiu
              Reporter:
              Serhii Lysenko
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Created:
                Updated:
                Resolved: