Andrew Wong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/15747 )
Change subject: test: fix HMS plugin test ...................................................................... test: fix HMS plugin test Running TestKuduMetastorePlugin with a version of Hive that includes 3ab174d82ffc2bd27432c0b04433be3bd7db5c6a raises a different exception than expected. This patch changes the test to set an appropiate location for Hive tables. Such a version of hive would yield errors like: 08:56:19.427 [ERROR - pool-11-thread-2] (RetryingHMSHandler.java:201) MetaException(message:An external table's location should not be located within managed warehouse root directory, table:table,location:/tmp/hive6879897156513669203/warehouse/table,managed warehouse:/tmp/hive6879897156513669203/warehouse) ...indicating the need for us to denote a different location. I went with the location used in d96f8fca55, which fixed a similar issue. Change-Id: I8ea8e3d6f2513f6ec3439b2e947e3564c4c53590 Reviewed-on: http://gerrit.cloudera.org:8080/15747 Reviewed-by: Grant Henke <[email protected]> Tested-by: Kudu Jenkins Reviewed-by: Hao Hao <[email protected]> --- M java/kudu-hive/src/test/java/org/apache/kudu/hive/metastore/TestKuduMetastorePlugin.java 1 file changed, 54 insertions(+), 20 deletions(-) Approvals: Grant Henke: Looks good to me, but someone else must approve Kudu Jenkins: Verified Hao Hao: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/15747 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I8ea8e3d6f2513f6ec3439b2e947e3564c4c53590 Gerrit-Change-Number: 15747 Gerrit-PatchSet: 3 Gerrit-Owner: Andrew Wong <[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)
