Tim Armstrong has uploaded a new change for review.

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

Change subject: Avoid unnecessary copy of RowDescriptor into RowBatch
......................................................................

Avoid unnecessary copy of RowDescriptor into RowBatch

This commit changes RowBatch::row_desc_ to be a const reference.
This is the pattern used elsewhere in the codebase and is safe
because the descriptor table will outlive any row batches for
the plan fragment.

Change-Id: I27c05090ee8b9dd9d650a5587a3ae42839aa0008
---
M be/src/runtime/descriptors.cc
M be/src/runtime/descriptors.h
M be/src/runtime/row-batch.h
3 files changed, 3 insertions(+), 3 deletions(-)


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I27c05090ee8b9dd9d650a5587a3ae42839aa0008
Gerrit-PatchSet: 1
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Tim Armstrong <[email protected]>

Reply via email to