Jordan Birdsell has submitted this change and it was merged.

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
Reviewed-on: http://gerrit.cloudera.org:8080/5018
Tested-by: Kudu Jenkins
Reviewed-by: Jordan Birdsell <[email protected]>
---
M python/kudu/client.pyx
M python/kudu/tests/test_client.py
2 files changed, 16 insertions(+), 12 deletions(-)

Approvals:
  Jordan Birdsell: Looks good to me, approved
  Kudu Jenkins: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I47be624de49d26de7a5c0b3af320e0bc907d70ad
Gerrit-PatchSet: 5
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <[email protected]>
Gerrit-Reviewer: Jordan Birdsell <[email protected]>
Gerrit-Reviewer: Kudu Jenkins

Reply via email to