Move part of PlanExplainerPipeline::getSummaryStats() to QE pipeline class

    • Type: Improvement
    • Resolution: Unresolved
    • Priority: Major - P3
    • None
    • Affects Version/s: None
    • Component/s: None
    • Query Execution
    • None
    • None
    • None
    • None
    • None
    • None
    • None

      The QE's Pipeline class already contains accumulatePlanSummaryStats() method which greatly resembles PlanExplainerPipeline::getSummaryStats().

      Having those methods side-by-side might help to understand the differences in handling the cursor stage and the nReturned counter.

      Another benefit of this ticket is to increase the encapsulation of the QE pipeline class by minimizing the number of getStages() calls.

      We might also check and document in the code if and why handling the cursor stage differently is necessary.

            Assignee:
            Unassigned
            Reporter:
            Romans Kasperovics
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated: