sentry-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From vam...@apache.org
Subject sentry git commit: SENTRY-1946: getPathsUpdatesFrom() got its boolean logic inversed which results in sending two snapshots at the same time (Vamsee Yarlagadda, Reviewed by: Alexander Kolbasov)
Date Tue, 12 Sep 2017 02:59:16 GMT
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 <vamsee@cloudera.com>
Authored: Mon Sep 11 17:43:24 2017 -0700
Committer: Vamsee Yarlagadda <vamsee@cloudera.com>
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();
     }


Mime
View raw message