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 <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Dan Burkert <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Todd Lipcon <[email protected]> Gerrit-Comment-Date: Mon, 22 Jun 2020 15:23:51 +0000 Gerrit-HasComments: No
