Mike Percy has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10968 )
Change subject: schema: add is_deleted virtual column ...................................................................... schema: add is_deleted virtual column This patch introduces a very basic concept of a "virtual column". Virtual columns borrow from other databases in that they are columns that, rather than being backed by physical data, are instead backed by Kudu itself. They may not be part of a schema during table creation/alteration, but may be added to projections during a scan. Kudu's virtual columns are defined as logical data types. As data types are not user-defined, there's no danger of a "collision" between a virtual column and a physical column as there would be if a virtual column occupied a well-defined name. A Kudu subsystem on the scan path that wishes to interact with a virtual column needs to first figure out if the projection includes it. When projected, the virtual column's data will be either some default or null (depending on exactly how it was defined in the projection); it's the subsystem's responsibility to fill in something meaningful afterwards. Beyond the basic definition, this patch introduces an IS_DELETED virtual column derived from BOOL. IS_DELETED will be used to support incremental backups by describing whether a row was deleted between two timestamps. Change-Id: I56eb1d44ba8bfbd76d8bb794b8076b695782939e Reviewed-on: http://gerrit.cloudera.org:8080/10968 Reviewed-by: Grant Henke <granthe...@apache.org> Reviewed-by: Alexey Serbin <aser...@cloudera.com> Tested-by: Adar Dembo <a...@cloudera.com> Reviewed-by: Mike Percy <mpe...@apache.org> --- M src/kudu/common/common.proto M src/kudu/common/schema.cc M src/kudu/common/types.cc M src/kudu/common/types.h M src/kudu/master/catalog_manager.cc M src/kudu/master/master-test.cc 6 files changed, 120 insertions(+), 27 deletions(-) Approvals: Grant Henke: Looks good to me, but someone else must approve Alexey Serbin: Looks good to me, but someone else must approve Adar Dembo: Verified Mike Percy: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/10968 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I56eb1d44ba8bfbd76d8bb794b8076b695782939e Gerrit-Change-Number: 10968 Gerrit-PatchSet: 16 Gerrit-Owner: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Dan Burkert <danburk...@apache.org> Gerrit-Reviewer: Grant Henke <granthe...@apache.org> Gerrit-Reviewer: Mike Percy <mpe...@apache.org> Gerrit-Reviewer: Tidy Bot Gerrit-Reviewer: Todd Lipcon <t...@apache.org>
