You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@impala.apache.org by "Tamas Mate (Jira)" <ji...@apache.org> on 2020/05/02 09:46:00 UTC

[jira] [Created] (IMPALA-9722) Consolidate unused total_width and the way avg_width is computed in PerColumnStats

Tamas Mate created IMPALA-9722:
----------------------------------

             Summary: Consolidate unused total_width and the way avg_width is computed in PerColumnStats
                 Key: IMPALA-9722
                 URL: https://issues.apache.org/jira/browse/IMPALA-9722
             Project: IMPALA
          Issue Type: Bug
          Components: Backend
            Reporter: Tamas Mate
            Assignee: Tamas Mate


PerColumnStats.total_width is an unused field in [incr-stats-util.cc|https://github.com/apache/impala/blob/master/be/src/exec/incr-stats-util.cc#L134] while the comment says that it is the result of {{(avg_width * num_rows)}}. Later [similar multiplication|https://github.com/apache/impala/blob/master/be/src/exec/incr-stats-util.cc#L159] can be found in the Update method, however the result is stored in {{avg_width}}.
{code:cpp}
avg_width += (new_avg_width * num_new_rows);
{code}
We should consolidate these fields and the calculations.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)