Csaba Ringhofer has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/15878 )
Change subject: IMPALA-9722: Consolidate avg_size calculation in PerColumnStats ...................................................................... IMPALA-9722: Consolidate avg_size calculation in PerColumnStats This change refactors the 'avg_size' calculation, 'total_width' will be storing the sum of row widths till a Finalize call calculates the average column size. Testing: - Added unit test to verify the aggregation result Change-Id: Iae1efb1c568c67dff6f25887c2ea2b8b249eea4b Reviewed-on: http://gerrit.cloudera.org:8080/15878 Tested-by: Impala Public Jenkins <[email protected]> Reviewed-by: Csaba Ringhofer <[email protected]> --- M be/src/exec/incr-stats-util-test.cc M be/src/exec/incr-stats-util.cc M be/src/exec/incr-stats-util.h 3 files changed, 27 insertions(+), 5 deletions(-) Approvals: Impala Public Jenkins: Verified Csaba Ringhofer: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/15878 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Iae1efb1c568c67dff6f25887c2ea2b8b249eea4b Gerrit-Change-Number: 15878 Gerrit-PatchSet: 3 Gerrit-Owner: Tamas Mate <[email protected]> Gerrit-Reviewer: Csaba Ringhofer <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
