impala-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Sailesh Mukil (Code Review)" <ger...@cloudera.org>
Subject [Impala-ASF-CR] IMPALA-5592: Fix computation of uncompressed row batch size
Date Fri, 14 Jul 2017 22:10:58 GMT
Sailesh Mukil has uploaded a new change for review.

  http://gerrit.cloudera.org:8080/7434

Change subject: IMPALA-5592: Fix computation of uncompressed row batch size
......................................................................

IMPALA-5592: Fix computation of uncompressed row batch size

The uncompressed row batch size is the size of a row batch
if the tuple data is sent without compression. It's computed
by the following formula:

serialized row batch size - compressed tuple data size + uncompressed tuple data size

The existing code is using the wrong field for the compressed tuple
data size and it causes uncompressed data size to be always identical
to the compressed row batch size. This change fixes the problem by
using the right field for the compressed tuple data size.

Change-Id: Ic8d285d1e4a1aa97c98c505351c18fea04c60311
---
M be/src/runtime/data-stream-sender.cc
1 file changed, 9 insertions(+), 5 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/34/7434/1
-- 
To view, visit http://gerrit.cloudera.org:8080/7434
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ic8d285d1e4a1aa97c98c505351c18fea04c60311
Gerrit-PatchSet: 1
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Sailesh Mukil <sailesh@cloudera.com>
Gerrit-Reviewer: Michael Ho <kwho@cloudera.com>

Mime
View raw message