Will Berkeley has uploaded a new change for review. http://gerrit.cloudera.org:8080/5132
Change subject: KUDU-1747 addColumn API is wrong wrt nullability and default values ...................................................................... KUDU-1747 addColumn API is wrong wrt nullability and default values Previously, the java client permitted adding - a non-null column with a default - a nullable column with no default but did not allow adding - a nullable column with a default even though it should be allowed. This patch adds that capability by adding a 'defaultVal' argument to an overloaded AlterTableOptions.addNullableColumn. It is backwards-compatible. Additionally, a test was added to check behavior of the three addColumn possibilities. Change-Id: I0706839a3dbe21532afd524e2d3fc020d31a9973 --- M java/kudu-client/src/main/java/org/apache/kudu/client/AlterTableOptions.java M java/kudu-client/src/test/java/org/apache/kudu/client/TestAlterTable.java 2 files changed, 43 insertions(+), 1 deletion(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/32/5132/1 -- To view, visit http://gerrit.cloudera.org:8080/5132 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I0706839a3dbe21532afd524e2d3fc020d31a9973 Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Will Berkeley <[email protected]>
