Hello Andrew Sherman, Impala Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/20732
to look at the new patch set (#3).
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
---
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(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/32/20732/3
--
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: newpatchset
Gerrit-Change-Id: I033ace83221bc5e0c84c19aac78a41c1eeda31f2
Gerrit-Change-Number: 20732
Gerrit-PatchSet: 3
Gerrit-Owner: Quanlong Huang <[email protected]>
Gerrit-Reviewer: Andrew Sherman <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
