Tim Armstrong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/14398 )
Change subject: IMPALA-8974: Fixed a bug when create kudu managed table without HMS config ...................................................................... IMPALA-8974: Fixed a bug when create kudu managed table without HMS config When catalogd connected to mysql/postgresql directly instead of use HMS, a kudu managerd table created DDL would failed due to Preconditions checked failed. The patch fixed the bug by using the if condition instead of the Preconditions functions which lead to exception when lack of HMS config. Tests: * Add test for create/drop kudu managerd table without HMS config * Ran all front-end tests Change-Id: Iacc53801a660c033869cb4747910c98a80e08297 Reviewed-on: http://gerrit.cloudera.org:8080/14398 Tested-by: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Reviewed-by: Tim Armstrong <tarmstr...@cloudera.com> --- M fe/src/main/java/org/apache/impala/catalog/KuduTable.java A fe/src/test/java/org/apache/impala/catalog/CreateKuduTableWithoutHMSTest.java M fe/src/test/resources/hive-site.xml.py 3 files changed, 144 insertions(+), 2 deletions(-) Approvals: Impala Public Jenkins: Verified Tim Armstrong: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/14398 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Iacc53801a660c033869cb4747910c98a80e08297 Gerrit-Change-Number: 14398 Gerrit-PatchSet: 21 Gerrit-Owner: wangsheng <sky...@163.com> Gerrit-Reviewer: Hao Hao <hao....@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Quanlong Huang <huangquanl...@gmail.com> Gerrit-Reviewer: Tim Armstrong <tarmstr...@cloudera.com> Gerrit-Reviewer: wangsheng <sky...@163.com>
