-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/58069/
-----------------------------------------------------------
(Updated April 4, 2017, 8:21 p.m.)
Review request for sentry, Hao Hao and kalyan kumar kalvagadda.
Changes
-------
Addressed code review comments from Hao Hao:
changed return type from Long to long for functions that get notification IDs
Bugs: SENTRY-1670
https://issues.apache.org/jira/browse/SENTRY-1670
Repository: sentry
Description
-------
SENTRY-1670 Expose current HMS notification ID as a Sentry gauge (metric)
Diffs (updated)
-----
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStore.java
bbfa713262c5b4d5cecccd95c823a78c9149752c
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/thrift/SentryMetrics.java
6ed2c781b4ac8819f6d17a249c33e32f0116e15a
Diff: https://reviews.apache.org/r/58069/diff/2/
Changes: https://reviews.apache.org/r/58069/diff/1-2/
Testing
-------
Thanks,
Alexander Kolbasov
