You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@sentry.apache.org by kalyan kumar kalvagadda via Review Board <no...@reviews.apache.org> on 2019/01/24 18:38:40 UTC

Re: Review Request 68622: SENTRY-2287: Find a way to report that HDFS ACL synchronization is complete.

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/68622/
-----------------------------------------------------------

(Updated Jan. 24, 2019, 6:38 p.m.)


Review request for sentry and Arjun Mishra.


Changes
-------

addressed review comments.


Bugs: SENTRY-2287
    https://issues.apache.org/jira/browse/SENTRY-2287


Repository: sentry


Description
-------

Currently there is no way to confirm that HDFS ACL synchronization is complete when snapshot is initiated. We need to identify that and log in console and log file as well.


Diffs (updated)
-----

  sentry-hdfs/sentry-hdfs-service/src/main/java/org/apache/sentry/hdfs/DBUpdateForwarder.java 08b16a4df3ea9126f21248365d6096fcdb83f21e 
  sentry-hdfs/sentry-hdfs-service/src/main/java/org/apache/sentry/hdfs/PathDeltaRetriever.java 81c614a345c122e067ec0a19b8f75766390b2ad4 
  sentry-hdfs/sentry-hdfs-service/src/main/java/org/apache/sentry/hdfs/PathImageRetriever.java 3532ef33dde8b119ab095861381fa52fa5520f4c 
  sentry-hdfs/sentry-hdfs-service/src/main/java/org/apache/sentry/hdfs/PermDeltaRetriever.java 8d6713acd61f99940f72c4985098dfeabb9fc832 
  sentry-hdfs/sentry-hdfs-service/src/main/java/org/apache/sentry/hdfs/PermImageRetriever.java 443434127b14fadaeb27717ad2370dcdc10ca70c 
  sentry-hdfs/sentry-hdfs-service/src/main/java/org/apache/sentry/hdfs/SentryHDFSServiceProcessor.java 0cd405b54ee2b4bf788dffb1ac606362614f6efe 
  sentry-service/sentry-service-server/src/main/java/org/apache/sentry/service/thrift/PathUpdaterState.java PRE-CREATION 
  sentry-service/sentry-service-server/src/main/java/org/apache/sentry/service/thrift/PermUpdaterState.java PRE-CREATION 
  sentry-service/sentry-service-server/src/main/java/org/apache/sentry/service/thrift/SentryStateBank.java 2afe919c31b0c5f495d54c6448593abad38eddc2 


Diff: https://reviews.apache.org/r/68622/diff/2/

Changes: https://reviews.apache.org/r/68622/diff/1-2/


Testing
-------


Thanks,

kalyan kumar kalvagadda