Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/20732 )
Change subject: IMPALA-12577: Correctly update last-synced-event-time ...................................................................... IMPALA-12577: Correctly update last-synced-event-time There are two metrics tracking the progress of HMS event processing: last-synced-event-id and last-synced-event-time. We have several places that only update last-synced-event-id without updating last-synced-event-time which is used to calculate the lag of event processing. Users might set up alerts on the lag so we'd better keep it updated. Tests - Add code to verify the metrics in MetastoreEventsProcessorTest - Fix a potential NullPointer error in HiveJdbcClientPool.java - Ran CORE test Change-Id: I033ace83221bc5e0c84c19aac78a41c1eeda31f2 Reviewed-on: http://gerrit.cloudera.org:8080/20732 Reviewed-by: Impala Public Jenkins <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M fe/src/main/java/org/apache/impala/catalog/events/MetastoreEventsProcessor.java M fe/src/test/java/org/apache/impala/catalog/events/SynchronousHMSEventProcessorForTests.java M fe/src/test/java/org/apache/impala/testutil/HiveJdbcClientPool.java 3 files changed, 71 insertions(+), 15 deletions(-) Approvals: Impala Public Jenkins: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/20732 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I033ace83221bc5e0c84c19aac78a41c1eeda31f2 Gerrit-Change-Number: 20732 Gerrit-PatchSet: 5 Gerrit-Owner: Quanlong Huang <[email protected]> Gerrit-Reviewer: Andrew Sherman <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Quanlong Huang <[email protected]>
