Grant Henke has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/15349 )

Change subject: [hms] Fix testAlterTableHandler in later Hive versions
......................................................................

[hms] Fix testAlterTableHandler in later Hive versions

This patch adjusts testAlterTableHandler to stop reusing
already created table objects when creating new tables.
This prevents issues if read-only properties are set.

An example of a read-only property is the ID field added in
HIVE-20556.

The error seen was:
`testAlterTableHandler(org.apache.kudu.hive.metastore.TestKuduMetastorePlugin)InvalidObjectException(message:Id
 shouldn't be set but table default.table has the Id set to 1. It's a read-only 
option)`

Change-Id: I8f8ddbe30a3e51957cafd8f1b596745b3262ca36
Reviewed-on: http://gerrit.cloudera.org:8080/15349
Reviewed-by: Hao Hao <[email protected]>
Tested-by: Grant Henke <[email protected]>
---
M 
java/kudu-hive/src/test/java/org/apache/kudu/hive/metastore/TestKuduMetastorePlugin.java
1 file changed, 7 insertions(+), 4 deletions(-)

Approvals:
  Hao Hao: Looks good to me, approved
  Grant Henke: Verified

--
To view, visit http://gerrit.cloudera.org:8080/15349
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I8f8ddbe30a3e51957cafd8f1b596745b3262ca36
Gerrit-Change-Number: 15349
Gerrit-PatchSet: 3
Gerrit-Owner: Grant Henke <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Grant Henke <[email protected]>
Gerrit-Reviewer: Hao Hao <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)

Reply via email to