You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sentry.apache.org by pr...@apache.org on 2014/09/23 20:12:31 UTC

git commit: SENTRY-396: The logic of Thrift multiplexedProcessor registers mutil processor isn't correct (guoquanshen via Prasad Mujumdar)

Repository: incubator-sentry
Updated Branches:
  refs/heads/master 48b883f4c -> ff19448ee


SENTRY-396: The logic of Thrift multiplexedProcessor registers mutil processor isn't correct (guoquanshen via Prasad Mujumdar)


Project: http://git-wip-us.apache.org/repos/asf/incubator-sentry/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-sentry/commit/ff19448e
Tree: http://git-wip-us.apache.org/repos/asf/incubator-sentry/tree/ff19448e
Diff: http://git-wip-us.apache.org/repos/asf/incubator-sentry/diff/ff19448e

Branch: refs/heads/master
Commit: ff19448eec13a121d658dde35b9bd46eeb3b7c31
Parents: 48b883f
Author: Prasad Mujumdar <pr...@cloudera.com>
Authored: Tue Sep 23 11:12:29 2014 -0700
Committer: Prasad Mujumdar <pr...@cloudera.com>
Committed: Tue Sep 23 11:12:29 2014 -0700

----------------------------------------------------------------------
 .../main/java/org/apache/sentry/service/thrift/SentryService.java | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-sentry/blob/ff19448e/sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/SentryService.java
----------------------------------------------------------------------
diff --git a/sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/SentryService.java b/sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/SentryService.java
index 6843e80..40e8a0e 100644
--- a/sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/SentryService.java
+++ b/sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/SentryService.java
@@ -182,8 +182,7 @@ public class SentryService implements Callable {
             .getConstructor(Configuration.class);
         ProcessorFactory factory = (ProcessorFactory) constructor
             .newInstance(conf);
-        registeredProcessor = registeredProcessor
-            || factory.register(processor);
+        registeredProcessor = factory.register(processor) || registeredProcessor;
       } catch (Exception e) {
         throw new IllegalStateException("Could not create "
             + processorFactory, e);