----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68547/ -----------------------------------------------------------
(Updated Sept. 28, 2018, 7:56 p.m.) Review request for sentry, kalyan kumar kalvagadda, Na Li, and Sergio Pena. Changes ------- Removed configuration and accommodated Kalyan's suggestions Bugs: SENTRY-2370 https://issues.apache.org/jira/browse/SENTRY-2370 Repository: sentry Description ------- When NN requests path updates from sentry and if it exceeds the time threshold, on consecutive attempts sentry will attempt to fetch the full update from scratch. Instead it should cache it and update the cache before sending it to NN Diffs (updated) ----- sentry-hdfs/sentry-hdfs-common/src/main/java/org/apache/sentry/hdfs/ImageCache.java PRE-CREATION sentry-hdfs/sentry-hdfs-service/src/main/java/org/apache/sentry/hdfs/DBUpdateForwarder.java 08b16a4df sentry-hdfs/sentry-hdfs-service/src/main/java/org/apache/sentry/hdfs/PathsUpdateImageCache.java PRE-CREATION sentry-hdfs/sentry-hdfs-service/src/main/java/org/apache/sentry/hdfs/PermissionsUpdateImageCache.java PRE-CREATION sentry-hdfs/sentry-hdfs-service/src/main/java/org/apache/sentry/hdfs/SentryPlugin.java b8f5ce7db sentry-hdfs/sentry-hdfs-service/src/test/java/org/apache/sentry/hdfs/TestDBUpdateForwarder.java f86ce6f83 Diff: https://reviews.apache.org/r/68547/diff/7/ Changes: https://reviews.apache.org/r/68547/diff/6-7/ Testing ------- Thanks, Arjun Mishra