Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/4310 to look at the new patch set (#6). Change subject: KUDU-861 Support changing default, storage attributes ...................................................................... KUDU-861 Support changing default, storage attributes This patch adds support for adding, changing, or removing column defaults. Changes to a column are encoded as a ColumnSchemaDelta, which can be merged with a ColumnSchema to change it. Handling default values is thorny, because, in both the C++ and Java clients, there's no type information available for the column at the time of the definition of the column alterations. In the short term, there's no problem, since changing and removing defaults works. In the long term, I think we should consider relocating alter column methods onto the table object instead of the client object. Changing type and nullability of a column is still unsupported (doing so is much more complicated). Change-Id: I457d99ba2188ef6e439df47c0d94f2dc1a62ea6c --- M java/kudu-client/src/main/java/org/apache/kudu/client/AlterTableOptions.java M java/kudu-client/src/main/java/org/apache/kudu/client/ProtobufHelper.java M java/kudu-client/src/test/java/org/apache/kudu/client/TestKuduTable.java M src/kudu/client/client-test.cc M src/kudu/client/schema.cc M src/kudu/client/schema.h M src/kudu/client/table_alterer-internal.cc M src/kudu/client/value-internal.h M src/kudu/client/value.cc M src/kudu/common/common.proto M src/kudu/common/schema.cc M src/kudu/common/schema.h M src/kudu/common/wire_protocol.cc M src/kudu/common/wire_protocol.h M src/kudu/integration-tests/alter_table-test.cc M src/kudu/master/catalog_manager.cc M src/kudu/master/master.proto 17 files changed, 788 insertions(+), 35 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/10/4310/6 -- To view, visit http://gerrit.cloudera.org:8080/4310 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: I457d99ba2188ef6e439df47c0d94f2dc1a62ea6c Gerrit-PatchSet: 6 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Will Berkeley <wdberke...@gmail.com> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Tidy Bot Gerrit-Reviewer: Todd Lipcon <t...@apache.org> Gerrit-Reviewer: Will Berkeley <wdberke...@gmail.com>