----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46081/ -----------------------------------------------------------
Review request for sentry. Repository: sentry Description ------- If processing of update on NN takes more than the cache interval time(default:30 secs), it is possible that two updates will be processed parallely leading to unexpected behavior. We should instead lock the processing of updates so that we ensure they happen sequentially. Also while doing so, we should add a flag to make sure we should never go into stale mode while processing an update. Diffs ----- sentry-hdfs/sentry-hdfs-common/src/main/java/org/apache/sentry/hdfs/UpdateableAuthzPaths.java 8fc5470088853e639e39a35fe930ca805f8ff08e sentry-hdfs/sentry-hdfs-namenode-plugin/src/main/java/org/apache/sentry/hdfs/SentryAuthorizationInfo.java c2416c1cf873484321cc7281bd46197d37a4195d sentry-hdfs/sentry-hdfs-namenode-plugin/src/main/java/org/apache/sentry/hdfs/UpdateableAuthzPermissions.java 33581b72369d6a46eefcabd0eb658e4213076c7c Diff: https://reviews.apache.org/r/46081/diff/ Testing ------- Thanks, Sravya Tirukkovalur