You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues-all@impala.apache.org by "Riza Suminto (Jira)" <ji...@apache.org> on 2024/05/30 21:51:00 UTC

[jira] [Created] (IMPALA-13119) CostingSegment.java is initialized with wrong cost

Riza Suminto created IMPALA-13119:
-------------------------------------

             Summary: CostingSegment.java is initialized with wrong cost
                 Key: IMPALA-13119
                 URL: https://issues.apache.org/jira/browse/IMPALA-13119
             Project: IMPALA
          Issue Type: Bug
          Components: Frontend
    Affects Versions: Impala 4.4.0
            Reporter: Riza Suminto
            Assignee: Riza Suminto


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.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

---------------------------------------------------------------------
To unsubscribe, e-mail: issues-all-unsubscribe@impala.apache.org
For additional commands, e-mail: issues-all-help@impala.apache.org