Hello Adar Dembo,

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

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

to review the following change.

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
---
M python/kudu/client.pyx
M python/kudu/tests/test_scanner.py
2 files changed, 5 insertions(+), 1 deletion(-)


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ic3c0e03caf76390497bcc464a1ebe508fbf33840
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: David Ribeiro Alves <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>

Reply via email to