-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/68025/
-----------------------------------------------------------
(Updated July 26, 2018, 3:03 p.m.)
Review request for sentry, kalyan kumar kalvagadda, Na Li, and Sergio Pena.
Changes
-------
Addressing Kalyan's comments
Bugs: SENTRY-2324
https://issues.apache.org/jira/browse/SENTRY-2324
Repository: sentry
Description
-------
Currently sentry fetches all notifications greater than latest notification Id
from HMS. With this sentry will only fetch notifications over a configured depth
Diffs (updated)
-----
sentry-hdfs/sentry-hdfs-common/src/main/java/org/apache/sentry/hdfs/ServiceConstants.java
a9afb151a
sentry-service/sentry-service-server/src/main/java/org/apache/sentry/provider/db/service/persistent/HMSFollower.java
fb826cfa6
sentry-service/sentry-service-server/src/main/java/org/apache/sentry/service/thrift/HiveNotificationFetcher.java
8490d7ac2
sentry-service/sentry-service-server/src/test/java/org/apache/sentry/provider/db/service/persistent/TestHMSFollower.java
1b4fa47dc
sentry-service/sentry-service-server/src/test/java/org/apache/sentry/service/thrift/TestHiveNotificationFetcher.java
83a1becd4
Diff: https://reviews.apache.org/r/68025/diff/2/
Changes: https://reviews.apache.org/r/68025/diff/1-2/
Testing
-------
$ mvn -f sentry-service/sentry-service-server/pom.xml test
Thanks,
Arjun Mishra
