Quanlong Huang has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/20567 )

Change subject: IMPALA-12483: Add new api in MetastoreServiceHandler for 
update_transaction_statistics method
......................................................................

IMPALA-12483: Add new api in MetastoreServiceHandler
for update_transaction_statistics method

In the MetastoreServiceHandler class, the
'update_transaction_statistics' API implementation is missing which is
causing test failures in the external front end integration tests.
This needs to be fixed by adding implementation of the
'update_transaction_statistics' api in MetastoreServiceHandler class.

This API was introduced in HMS long ago but only recently we
encountered a bug with this missing API when testing materialized views
with transactional properties in the external front-end setup.

Testing:
Added a test in the 'test_hms_apis()' to verify that the
'update_transaction_statistics' api in HMS is reachable from impala.

Change-Id: Icd9bc47ff8c6e5e7b31961d7ee6b8c1d4bd74870
Reviewed-on: http://gerrit.cloudera.org:8080/20567
Tested-by: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Reviewed-by: Quanlong Huang <huangquanl...@gmail.com>
---
M 
fe/src/main/java/org/apache/impala/catalog/metastore/MetastoreServiceHandler.java
M tests/custom_cluster/test_metastore_service.py
2 files changed, 22 insertions(+), 0 deletions(-)

Approvals:
  Impala Public Jenkins: Verified
  Quanlong Huang: Looks good to me, approved

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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: Icd9bc47ff8c6e5e7b31961d7ee6b8c1d4bd74870
Gerrit-Change-Number: 20567
Gerrit-PatchSet: 3
Gerrit-Owner: Sai Hemanth Gantasala <saihema...@cloudera.com>
Gerrit-Reviewer: Csaba Ringhofer <csringho...@cloudera.com>
Gerrit-Reviewer: Daniel Becker <daniel.bec...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Quanlong Huang <huangquanl...@gmail.com>
Gerrit-Reviewer: Sai Hemanth Gantasala <saihema...@cloudera.com>

Reply via email to