Hello Wes McKinney, Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/2031
to look at the new patch set (#2).
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/2
--
To view, visit http://gerrit.cloudera.org:8080/2031
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ic3c0e03caf76390497bcc464a1ebe508fbf33840
Gerrit-PatchSet: 2
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: David Ribeiro Alves <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <[email protected]>
Gerrit-Reviewer: Wes McKinney <[email protected]>
