You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "Todd Lipcon (Code Review)" <ge...@cloudera.org> on 2017/01/04 19:47:45 UTC

[kudu-CR] KUDU-1760. Add test coverage of reading pre-REINSERT after ALTER

Hello David Ribeiro Alves, Kudu Jenkins,

I'd like you to reexamine a change.  Please visit

    http://gerrit.cloudera.org:8080/5488

to look at the new patch set (#2).

Change subject: KUDU-1760. Add test coverage of reading pre-REINSERT after ALTER
......................................................................

KUDU-1760. Add test coverage of reading pre-REINSERT after ALTER

This adds a test for the case where an ALTER TABLE adds a column, and
then we read the old history of that row from prior to the addition of
the column. We make sure that the default value of the new column is
returned.

Change-Id: Id7d44f6e2fe4d3cae03e0024c70e9f2ae84b614c
---
M src/kudu/integration-tests/alter_table-test.cc
M src/kudu/tablet/compaction.cc
2 files changed, 47 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/88/5488/2
-- 
To view, visit http://gerrit.cloudera.org:8080/5488
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Id7d44f6e2fe4d3cae03e0024c70e9f2ae84b614c
Gerrit-PatchSet: 2
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: David Ribeiro Alves <dr...@apache.org>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy <mp...@apache.org>
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>