IMPALA-3179: Fix alter table properties for Kudu tables This is one of the merge follow up tasks. It seems like there was just a line missing to copy the metastore data into the Kudu table object. The HDFS table class does the same thing as in this change.
Change-Id: I51c9942f2f398afb7dff2485da759a185ad7505f Reviewed-on: http://gerrit.cloudera.org:8080/2728 Reviewed-by: Casey Ching <[email protected]> Tested-by: Internal Jenkins Project: http://git-wip-us.apache.org/repos/asf/incubator-impala/repo Commit: http://git-wip-us.apache.org/repos/asf/incubator-impala/commit/0d9028dd Tree: http://git-wip-us.apache.org/repos/asf/incubator-impala/tree/0d9028dd Diff: http://git-wip-us.apache.org/repos/asf/incubator-impala/diff/0d9028dd Branch: refs/heads/master Commit: 0d9028dd49af9b371d40aa91ea3c201f63454d1a Parents: 4bdd0b9 Author: casey <[email protected]> Authored: Thu Mar 31 17:04:48 2016 -0700 Committer: Tim Armstrong <[email protected]> Committed: Tue Apr 12 14:03:44 2016 -0700 ---------------------------------------------------------------------- fe/src/main/java/com/cloudera/impala/catalog/KuduTable.java | 1 + tests/query_test/test_kudu.py | 7 +++---- 2 files changed, 4 insertions(+), 4 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/incubator-impala/blob/0d9028dd/fe/src/main/java/com/cloudera/impala/catalog/KuduTable.java ---------------------------------------------------------------------- diff --git a/fe/src/main/java/com/cloudera/impala/catalog/KuduTable.java b/fe/src/main/java/com/cloudera/impala/catalog/KuduTable.java index 2b60766..541908a 100644 --- a/fe/src/main/java/com/cloudera/impala/catalog/KuduTable.java +++ b/fe/src/main/java/com/cloudera/impala/catalog/KuduTable.java @@ -179,6 +179,7 @@ public class KuduTable extends Table { "Cannot load Kudu table %s, table is corrupt.", name_)); } + msTable_ = msTbl; kuduTableName_ = msTbl.getParameters().get(KEY_TABLE_NAME); kuduMasters_ = msTbl.getParameters().get(KEY_MASTER_ADDRESSES); http://git-wip-us.apache.org/repos/asf/incubator-impala/blob/0d9028dd/tests/query_test/test_kudu.py ---------------------------------------------------------------------- diff --git a/tests/query_test/test_kudu.py b/tests/query_test/test_kudu.py index d5e4555..25e7d95 100644 --- a/tests/query_test/test_kudu.py +++ b/tests/query_test/test_kudu.py @@ -82,10 +82,9 @@ class TestKuduOperations(ImpalaTestSuite): def test_kudu_partition_ddl(self, vector): self.run_test_case('QueryTest/kudu_partition_ddl', vector, use_db="kududb_test") - # TODO(kudu-merge) IMPALA-3179 - Altering table properties is broken. When that is - # solved uncomment this. - #def test_kudu_alter_table(self, vector): - # self.run_test_case('QueryTest/kudu_alter', vector, use_db="kududb_test") + @pytest.mark.execute_serially + def test_kudu_alter_table(self, vector): + self.run_test_case('QueryTest/kudu_alter', vector, use_db="kududb_test") def test_kudu_stats(self, vector): self.run_test_case('QueryTest/kudu_stats', vector, use_db="kududb_test")
