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 <[email protected]> Authored: Tue Sep 23 11:12:29 2014 -0700 Committer: Prasad Mujumdar <[email protected]> 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);
