Dan Burkert has posted comments on this change.

Change subject: KUDU-1811: C++ client: use larger batches when fetching scan 
tokens
......................................................................


Patch Set 5:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/7748/5/src/kudu/client/meta_cache.h
File src/kudu/client/meta_cache.h:

Line 390:   void LookupTabletsByKeyOrNext(const KuduTable* table,
> Thanks for the comments. I thought about it but went this way to make the C
I don't quite follow - from looking at the C++ implementation, it's always 
using the default constant value of 10 (kMaxReturnedTableLocations).


-- 
To view, visit http://gerrit.cloudera.org:8080/7748
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I79340dc9963944454770d82a2fbaba1b0c8a810e
Gerrit-PatchSet: 5
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Jun He <junhe...@gmail.com>
Gerrit-Reviewer: Dan Burkert <danburk...@apache.org>
Gerrit-Reviewer: Jun He <junhe...@gmail.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Tidy Bot
Gerrit-Reviewer: Todd Lipcon <t...@apache.org>
Gerrit-HasComments: Yes

Reply via email to