-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62096/
-----------------------------------------------------------
(Updated Sept. 5, 2017, 10:41 p.m.)
Review request for sentry, Alexander Kolbasov, Brian Towles, Na Li, and Vamsee
Yarlagadda.
Changes
-------
Renamed the atomic flag to pathsRetrieverBusy
Bugs: sentry-1919
https://issues.apache.org/jira/browse/sentry-1919
Repository: sentry
Description
-------
The patch uses an atomic boolean flag that prevents multiple HDFS requests to
get the paths updates (either deltas or full images) by returning an empty
paths updates list if the flag is set as not available.
The reason to prevent for deltas as well is because the code change was much
easier to do in once place (SentryHDFSServiceProcessor). We shouldn't get any
issues if we do it this way (unless the reviewer thinks otherwise).
Diffs (updated)
-----
sentry-hdfs/sentry-hdfs-service/src/main/java/org/apache/sentry/hdfs/SentryHDFSServiceProcessor.java
6221f3d01
Diff: https://reviews.apache.org/r/62096/diff/2/
Changes: https://reviews.apache.org/r/62096/diff/1-2/
Testing
-------
Thanks,
Sergio Pena