Hello Jordan Birdsell,

I'd like you to do a code review.  Please visit

    http://gerrit.cloudera.org:8080/5018

to review the following change.

Change subject: python: check status result when setting row contents
......................................................................

python: check status result when setting row contents

The various PartialRow::Set*() functions return a Status to indicate an
error if an incorrect type is passed or when trying to set a NULL value
on a non-NULL column. We previously weren't checking these Status
results in the Python client.

Change-Id: I47be624de49d26de7a5c0b3af320e0bc907d70ad
---
M python/kudu/client.pyx
M python/kudu/tests/test_client.py
2 files changed, 13 insertions(+), 10 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/18/5018/1
-- 
To view, visit http://gerrit.cloudera.org:8080/5018
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I47be624de49d26de7a5c0b3af320e0bc907d70ad
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <t...@apache.org>
Gerrit-Reviewer: Jordan Birdsell <jtbirds...@apache.org>

Reply via email to