Uploaded image for project: 'IMPALA'
  1. IMPALA
  2. IMPALA-13119

CostingSegment.java is initialized with wrong cost

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • Impala 4.4.0
    • Impala 4.5.0
    • Frontend
    • None

    Description

      CostingSegment.java has two public constructor: one accept PlanNode, while the other accept DataSink as parameter. Both call appendCost method, which sum the additionalCost with the segment's current cost_.

      However, if cost_ were ProcessingCost.zero(), it can mistakenly setNumRowToConsume to 0.

      https://github.com/apache/impala/blob/master/fe/src/main/java/org/apache/impala/planner/CostingSegment.java#L114 

      The public constructor should just initialize cost_ with ProcessingCost from PlanNode or DataSink from constructor.

      Attachments

        Activity

          People

            rizaon Riza Suminto
            rizaon Riza Suminto
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: