Todd Lipcon has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9783 )
Change subject: KUDU-16 pt 1: add server-side limits on scanners ...................................................................... KUDU-16 pt 1: add server-side limits on scanners This patch adds the functionality to specify limits to the number of rows returned per tablet server scanner. Internally, each server-side scanner now tracks the number of rows it has already returned and adjusts the number of rows to serialize per batch based on this limit. Setting a non-positive limit is grounds for short circuiting the scan. Note: Use of this functionality by the client will come in a later patch. Change-Id: I3ca5ddff09f4910d12f80259bd11e4027f99aa7c Reviewed-on: http://gerrit.cloudera.org:8080/9783 Tested-by: Kudu Jenkins Reviewed-by: Todd Lipcon <[email protected]> --- M src/kudu/common/rowblock.cc M src/kudu/common/rowblock.h M src/kudu/common/scan_spec.cc M src/kudu/common/scan_spec.h M src/kudu/tserver/scanners.cc M src/kudu/tserver/scanners.h M src/kudu/tserver/tablet_server-test.cc M src/kudu/tserver/tablet_service.cc M src/kudu/tserver/tserver.proto 9 files changed, 184 insertions(+), 32 deletions(-) Approvals: Kudu Jenkins: Verified Todd Lipcon: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/9783 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I3ca5ddff09f4910d12f80259bd11e4027f99aa7c Gerrit-Change-Number: 9783 Gerrit-PatchSet: 16 Gerrit-Owner: Andrew Wong <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Dan Burkert <[email protected]> Gerrit-Reviewer: David Ribeiro Alves <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Tidy Bot Gerrit-Reviewer: Todd Lipcon <[email protected]>
