Internal Jenkins has submitted this change and it was merged.

Change subject: Remove redundant row copy in DataStreamSender
......................................................................


Remove redundant row copy in DataStreamSender

RowBatch::CopyRow() does a shallow copy of tuple pointers, which is
redundant with the tuple pointer copies in the loop immediately
after it.

Change-Id: Ic6603624cb2abc5c8de6f15c9ced73dc3b25a893
Reviewed-on: http://gerrit.cloudera.org:8080/2371
Reviewed-by: Tim Armstrong <[email protected]>
Tested-by: Internal Jenkins
---
M be/src/runtime/data-stream-sender.cc
1 file changed, 1 insertion(+), 3 deletions(-)

Approvals:
  Internal Jenkins: Verified
  Tim Armstrong: Looks good to me, approved



-- 
To view, visit http://gerrit.cloudera.org:8080/2371
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ic6603624cb2abc5c8de6f15c9ced73dc3b25a893
Gerrit-PatchSet: 5
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: Tim Armstrong <[email protected]>
Gerrit-Reviewer: Alex Behm <[email protected]>
Gerrit-Reviewer: Internal Jenkins
Gerrit-Reviewer: Marcel Kornacker <[email protected]>
Gerrit-Reviewer: Michael Ho <[email protected]>
Gerrit-Reviewer: Skye Wanderman-Milne <[email protected]>
Gerrit-Reviewer: Tim Armstrong <[email protected]>

Reply via email to