Jordan Birdsell has posted comments on this change.

Change subject: KUDU-1304 - [python] Enable listing of tablet servers
......................................................................


Patch Set 3:

(5 comments)

http://gerrit.cloudera.org:8080/#/c/4328/2/python/kudu/client.pyx
File python/kudu/client.pyx:

PS2, Line 368: ts = TabletServer()
             :             result.append(ts._init(tservers[i])
> I assume this 'init' thing is because it's not possible to do a normal __in
Yep, annoying, seemed like the only other option was to deconstruct/reconstruct 
the class altogether but that seemed like a worse idea.


PS2, Line 499: Repres
> typo
Done


PS2, Line 500: TabletSe
> expand out the abbreviation here 'TabletServers'
Done


PS2, Line 507: _init
> does this show up as a method in the generated python class? If so, maybe i
Done


Line 510: 
> I think we need some __dealloc__ here which takes care of deleting _tserver
Done


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I51794168750b358d8c1b371ac2bd08b5c8f15395
Gerrit-PatchSet: 3
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Jordan Birdsell <jordantbirds...@gmail.com>
Gerrit-Reviewer: Jordan Birdsell <jordantbirds...@gmail.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <t...@apache.org>
Gerrit-HasComments: Yes

Reply via email to