----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55706/ -----------------------------------------------------------
(Updated March 2, 2017, 1:49 a.m.) Review request for sentry. Repository: sentry Description ------- Read full permission and path snapshot from SentryDB and make the update available for NN plugin upon requests. In detail: 1. Added Path/PermissionImage classes to represent corresponding Path/Permission snapshot read from DB. 2. Refactor full snapshot retriever APIs in SentryStore to become a single transaction to ensure snapshot consistency. 3. Added Path/PermissionImageRetriever classes to retrieve Path/PermissionImage from DB and convert to corresponding Path/PermissionUpdate, which later would be consumed by NN plugin. Diffs (updated) ----- sentry-hdfs/sentry-hdfs-common/src/main/java/org/apache/sentry/hdfs/ImageRetriever.java PRE-CREATION sentry-hdfs/sentry-hdfs-common/src/main/java/org/apache/sentry/hdfs/PathsUpdate.java ffb07560d7c40fb14cef77950012c2c81fa3fd28 sentry-hdfs/sentry-hdfs-service/src/main/java/org/apache/sentry/hdfs/PathImageRetriever.java PRE-CREATION sentry-hdfs/sentry-hdfs-service/src/main/java/org/apache/sentry/hdfs/PermImageRetriever.java PRE-CREATION sentry-hdfs/sentry-hdfs-service/src/main/java/org/apache/sentry/hdfs/SentryHdfsMetricsUtil.java e68c7080cec007e3cdf2c89efb9701eef9cca0a0 sentry-hdfs/sentry-hdfs-service/src/main/java/org/apache/sentry/hdfs/SentryPlugin.java f68f690436301c79e93b0742996dda25a750c0c3 sentry-hdfs/sentry-hdfs-service/src/main/java/org/apache/sentry/hdfs/UpdateForwarder.java 6d5c607273bb08597780b655d7b59cd41f0844bb sentry-hdfs/sentry-hdfs-service/src/main/java/org/apache/sentry/hdfs/UpdateablePermissions.java fe2baa6a446874185e8344bb16d76d803826d1f3 sentry-hdfs/sentry-hdfs-service/src/test/java/org/apache/sentry/hdfs/TestUpdateForwarder.java 0f0d0a743d5a5f9f1678ac7c5217b9a27537e85b sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/PathsImage.java PRE-CREATION sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/PermissionsImage.java PRE-CREATION sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStore.java b9272bc80ea473685b90f6c30dc8e0b0dff7b9a9 sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/persistent/TestSentryStore.java 0e227551c3d105909fc9866134bbd7bcf2cf668f Diff: https://reviews.apache.org/r/55706/diff/3/ Changes: https://reviews.apache.org/r/55706/diff/2-3/ Testing ------- Thanks, Hao Hao
