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

Consolidate unused total_width and the way avg_width is computed in PerColumnStats

Attach filesAttach ScreenshotVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Bug
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • None
    • Impala 4.0.0
    • Backend
    • ghx-label-8

    Description

      PerColumnStats.total_width is an unused field in incr-stats-util.cc while the comment says that it is the result of (avg_width * num_rows). Later similar multiplication can be found in the Update method, however the result is stored in avg_width.

      avg_width += (new_avg_width * num_new_rows);
      

      We should consolidate these fields and the calculations.

      Attachments

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            tmate Tamas Mate
            tmate Tamas Mate
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment