Andrew Wong has uploaded this change for review. ( http://gerrit.cloudera.org:8080/9923
Change subject: WIP KUDU-16 pt 3: add per-scan-token limits ...................................................................... WIP KUDU-16 pt 3: add per-scan-token limits This patch implements scan-limiting on a per-token basis. Kudu only exposes a builder API for batches of tokens. These tokens get sent out and instantiate scanners with any specified limit. This means that if the builder limit is l, with t tokens, the number of rows returned will be at most l * t. This is preferable to splitting the limit across tokens, e.g. by limiting each token to l / t rows, as this may yield fewer than l rows across tokens, when more than l rows actually exist, which doesn't seem desirable. WIP Maybe we should expose setting individual token limits. Also I'm not sure this is actually a useful API or the one that we want to expose. Change-Id: I655f07f10a9a99e9766402729361de97e929136a --- M src/kudu/client/client.cc M src/kudu/client/client.h M src/kudu/client/scan_token-internal.cc M src/kudu/client/scan_token-test.cc 4 files changed, 37 insertions(+), 1 deletion(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/23/9923/1 -- To view, visit http://gerrit.cloudera.org:8080/9923 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I655f07f10a9a99e9766402729361de97e929136a Gerrit-Change-Number: 9923 Gerrit-PatchSet: 1 Gerrit-Owner: Andrew Wong <[email protected]>
