Repository: sentry Updated Branches: refs/heads/master b38267765 -> 342025640
SENTRY-1946: getPathsUpdatesFrom() got its boolean logic inversed which results in sending two snapshots at the same time (Vamsee Yarlagadda, Reviewed by: Alexander Kolbasov) Project: http://git-wip-us.apache.org/repos/asf/sentry/repo Commit: http://git-wip-us.apache.org/repos/asf/sentry/commit/34202564 Tree: http://git-wip-us.apache.org/repos/asf/sentry/tree/34202564 Diff: http://git-wip-us.apache.org/repos/asf/sentry/diff/34202564 Branch: refs/heads/master Commit: 342025640a61567df3a7628b701efb1c9590ef24 Parents: b382677 Author: Vamsee Yarlagadda <[email protected]> Authored: Mon Sep 11 17:43:24 2017 -0700 Committer: Vamsee Yarlagadda <[email protected]> Committed: Mon Sep 11 19:58:29 2017 -0700 ---------------------------------------------------------------------- .../java/org/apache/sentry/hdfs/SentryHDFSServiceProcessor.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/sentry/blob/34202564/sentry-hdfs/sentry-hdfs-service/src/main/java/org/apache/sentry/hdfs/SentryHDFSServiceProcessor.java ---------------------------------------------------------------------- diff --git a/sentry-hdfs/sentry-hdfs-service/src/main/java/org/apache/sentry/hdfs/SentryHDFSServiceProcessor.java b/sentry-hdfs/sentry-hdfs-service/src/main/java/org/apache/sentry/hdfs/SentryHDFSServiceProcessor.java index 2866ab3..9041ff6 100644 --- a/sentry-hdfs/sentry-hdfs-service/src/main/java/org/apache/sentry/hdfs/SentryHDFSServiceProcessor.java +++ b/sentry-hdfs/sentry-hdfs-service/src/main/java/org/apache/sentry/hdfs/SentryHDFSServiceProcessor.java @@ -121,7 +121,7 @@ public class SentryHDFSServiceProcessor implements SentryHDFSService.Iface { // If another paths updates request is in progress by another HDFS NameNode, then we // return an empty image for now to avoid a large memory consumption - if (pathsRetrieverBusy.compareAndSet(false, true)) { + if (!pathsRetrieverBusy.compareAndSet(false, true)) { LOGGER.debug("PATHS updates are not available because another request is in progress."); return Collections.emptyList(); }
