David Ribeiro Alves has submitted this change and it was merged. 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 <[email protected]> Tested-by: Kudu Jenkins Reviewed-by: Todd Lipcon <[email protected]> --- M python/kudu/client.pyx M python/kudu/tests/test_scanner.py 2 files changed, 6 insertions(+), 1 deletion(-) Approvals: David Ribeiro Alves: Looks good to me, but someone else must approve Todd Lipcon: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/2031 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic3c0e03caf76390497bcc464a1ebe508fbf33840 Gerrit-PatchSet: 5 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: David Ribeiro Alves <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: David Ribeiro Alves <[email protected]> Gerrit-Reviewer: Jean-Daniel Cryans Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <[email protected]> Gerrit-Reviewer: Wes McKinney <[email protected]>
