Todd Lipcon has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/9924 )

Change subject: WIP KUDU-16 pt 4: python client support for limits
......................................................................


Patch Set 1:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/9924/1/python/kudu/libkudu_client.pxd
File python/kudu/libkudu_client.pxd:

http://gerrit.cloudera.org:8080/#/c/9924/1/python/kudu/libkudu_client.pxd@681
PS1, Line 681: (
nit: weird spacing


http://gerrit.cloudera.org:8080/#/c/9924/1/python/kudu/libkudu_client.pxd@714
PS1, Line 714:         Status SetLimitPerToken(int64_t limit)
maybe break this into a separate commit?



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I49e24ff45c7dd359b400e4270fe7573305075b39
Gerrit-Change-Number: 9924
Gerrit-PatchSet: 1
Gerrit-Owner: Andrew Wong <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <[email protected]>
Gerrit-Comment-Date: Wed, 04 Apr 2018 19:27:02 +0000
Gerrit-HasComments: Yes

Reply via email to