You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "Grant Henke (Code Review)" <ge...@cloudera.org> on 2020/06/22 15:23:51 UTC

[kudu-CR] WIP KUDU-16 pt 3: add per-scan-token limits

Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/9923 )

Change subject: WIP KUDU-16 pt 3: add per-scan-token limits
......................................................................


Patch Set 1:

Revisiting this. I was a bit surprised to see the limit used in the Java client, but not in the C++ client. Given the limit is already defined and used on the scan token it seams like plumbing it through cant hurt and would make the client implementations more consistent.


-- 
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: comment
Gerrit-Change-Id: I655f07f10a9a99e9766402729361de97e929136a
Gerrit-Change-Number: 9923
Gerrit-PatchSet: 1
Gerrit-Owner: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Dan Burkert <da...@apache.org>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
Gerrit-Comment-Date: Mon, 22 Jun 2020 15:23:51 +0000
Gerrit-HasComments: No