You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cassandra.apache.org by "Jeremiah Jordan (JIRA)" <ji...@apache.org> on 2016/08/18 21:13:20 UTC

[jira] [Commented] (CASSANDRA-9054) Let DatabaseDescriptor not implicitly startup services

    [ https://issues.apache.org/jira/browse/CASSANDRA-9054?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15427156#comment-15427156 ] 

Jeremiah Jordan commented on CASSANDRA-9054:
--------------------------------------------

Was just looking through this.
{{AuthConfig.applyAuthz()}} is called from both {{DatabaseDescriptor.daemonInitialization()}} and {{CassandraDaemon.applyConfig()}}, which means it gets called twice, as {{applyConfig}} also calls {{daemonInitialization}}

Also a nit on naming {{applyAuthz} should probably just be called {{apply}} or {{applyAuth}} we usually use "Authz" to mean specifically Authori*z*ation, and that method sets up both Authorization and Authentication.

> Let DatabaseDescriptor not implicitly startup services
> ------------------------------------------------------
>
>                 Key: CASSANDRA-9054
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-9054
>             Project: Cassandra
>          Issue Type: Improvement
>            Reporter: Jeremiah Jordan
>            Assignee: Robert Stupp
>             Fix For: 3.10
>
>
> Right now to get at Config stuff you go through DatabaseDescriptor.  But when you instantiate DatabaseDescriptor it actually opens system tables and such, which triggers commit log replays, and other things if the right flags aren't set ahead of time.  This makes getting at config stuff from tools annoying, as you have to be very careful about instantiation orders.
> It would be nice if we could break DatabaseDescriptor up into multiple classes, so that getting at config stuff from tools wasn't such a pain.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)