You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@kudu.apache.org by "David Ribeiro Alves (Code Review)" <ge...@ip-10-146-233-104.ec2.internal> on 2016/02/05 21:41:59 UTC

[kudu-CR](branch-0.7.0) [python] - On writes, allow to set values to None

David Ribeiro Alves has uploaded a new change for review.

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

Change subject: [python] - On writes, allow to set values to None
......................................................................

[python] - On writes, allow to set values to None

Wes pointed out that we weren't testing this case and it turns out that
it didn't work. We would crash in that case as we weren't checking for None,
which this patch now does.

Change-Id: Ic3c0e03caf76390497bcc464a1ebe508fbf33840
Reviewed-on: http://gerrit.cloudera.org:8080/2031
Reviewed-by: David Ribeiro Alves <da...@cloudera.com>
Tested-by: Kudu Jenkins
Reviewed-by: Todd Lipcon <to...@apache.org>
(cherry picked from commit 6d74508a922b8d3d8ba0518b5eb63acd10c222ae)
---
M python/kudu/client.pyx
M python/kudu/tests/test_scanner.py
2 files changed, 6 insertions(+), 1 deletion(-)


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ic3c0e03caf76390497bcc464a1ebe508fbf33840
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: branch-0.7.0
Gerrit-Owner: David Ribeiro Alves <da...@cloudera.com>