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);