You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@impala.apache.org by "Tim Armstrong (Code Review)" <ge...@cloudera.org> on 2018/07/16 22:07:21 UTC

[Impala-ASF-CR] cleanup: extract RowBatchQueue into its own file

Tim Armstrong has uploaded a new patch set (#2). ( http://gerrit.cloudera.org:8080/10943 )

Change subject: cleanup: extract RowBatchQueue into its own file
......................................................................

cleanup: extract RowBatchQueue into its own file

While looking at IMPALA-7096, I noticed that RowBatchQueue was
implemented in a strange place.

Change-Id: I3577c1c6920b8cf858c8d49f8812ccc305d833f6
---
M be/src/exec/blocking-join-node.h
M be/src/exec/exec-node.cc
M be/src/exec/exec-node.h
M be/src/exec/hdfs-scan-node.cc
M be/src/exec/kudu-scan-node.cc
M be/src/exec/scan-node.cc
M be/src/exec/scan-node.h
M be/src/exec/scanner-context.cc
M be/src/runtime/CMakeLists.txt
M be/src/runtime/data-stream-recvr.cc
M be/src/runtime/krpc-data-stream-recvr.cc
A be/src/runtime/row-batch-queue.cc
A be/src/runtime/row-batch-queue.h
13 files changed, 129 insertions(+), 66 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/43/10943/2
-- 
To view, visit http://gerrit.cloudera.org:8080/10943
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I3577c1c6920b8cf858c8d49f8812ccc305d833f6
Gerrit-Change-Number: 10943
Gerrit-PatchSet: 2
Gerrit-Owner: Tim Armstrong <ta...@cloudera.com>