You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cassandra.apache.org by jb...@apache.org on 2015/08/31 19:04:59 UTC
[1/6] cassandra git commit: Defer initialization of default
RoleManager until IAuthenticator is set reviewed by jbellis for
CASSANDRA-10209
Repository: cassandra
Updated Branches:
refs/heads/cassandra-2.2 c870adc2f -> 0d2f4038f
refs/heads/cassandra-3.0 0828e2499 -> 3bc585d5d
refs/heads/trunk ae7afb7a8 -> 5d0c6ef77
Defer initialization of default RoleManager until IAuthenticator is set
reviewed by jbellis for CASSANDRA-10209
Project: http://git-wip-us.apache.org/repos/asf/cassandra/repo
Commit: http://git-wip-us.apache.org/repos/asf/cassandra/commit/0d2f4038
Tree: http://git-wip-us.apache.org/repos/asf/cassandra/tree/0d2f4038
Diff: http://git-wip-us.apache.org/repos/asf/cassandra/diff/0d2f4038
Branch: refs/heads/cassandra-2.2
Commit: 0d2f4038fc2564c695d0d8c4b1ad523ec1afaceb
Parents: c870adc
Author: Sam Tunnicliffe <sa...@beobal.com>
Authored: Thu Aug 27 20:21:17 2015 +0100
Committer: Jonathan Ellis <jb...@apache.org>
Committed: Mon Aug 31 12:04:01 2015 -0500
----------------------------------------------------------------------
.../org/apache/cassandra/config/DatabaseDescriptor.java | 9 ++++++---
1 file changed, 6 insertions(+), 3 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/cassandra/blob/0d2f4038/src/java/org/apache/cassandra/config/DatabaseDescriptor.java
----------------------------------------------------------------------
diff --git a/src/java/org/apache/cassandra/config/DatabaseDescriptor.java b/src/java/org/apache/cassandra/config/DatabaseDescriptor.java
index b7e3eaa..9cf3959 100644
--- a/src/java/org/apache/cassandra/config/DatabaseDescriptor.java
+++ b/src/java/org/apache/cassandra/config/DatabaseDescriptor.java
@@ -87,7 +87,9 @@ public class DatabaseDescriptor
private static IAuthenticator authenticator = new AllowAllAuthenticator();
private static IAuthorizer authorizer = new AllowAllAuthorizer();
- private static IRoleManager roleManager = new CassandraRoleManager();
+ // Don't initialize the role manager until applying config. The options supported by CassandraRoleManager
+ // depend on the configured IAuthenticator, so defer creating it until that's been set.
+ private static IRoleManager roleManager;
private static IRequestScheduler requestScheduler;
private static RequestSchedulerId requestSchedulerId;
@@ -323,8 +325,9 @@ public class DatabaseDescriptor
if (authenticator instanceof AllowAllAuthenticator && !(authorizer instanceof AllowAllAuthorizer))
throw new ConfigurationException("AllowAllAuthenticator can't be used with " + conf.authorizer, false);
- if (conf.role_manager != null)
- roleManager = FBUtilities.newRoleManager(conf.role_manager);
+ roleManager = conf.role_manager == null
+ ? new CassandraRoleManager()
+ : FBUtilities.newRoleManager(conf.role_manager);
if (authenticator instanceof PasswordAuthenticator && !(roleManager instanceof CassandraRoleManager))
throw new ConfigurationException("CassandraRoleManager must be used with PasswordAuthenticator", false);
[4/6] cassandra git commit: Merge branch 'cassandra-2.2' into
cassandra-3.0
Posted by jb...@apache.org.
Merge branch 'cassandra-2.2' into cassandra-3.0
Project: http://git-wip-us.apache.org/repos/asf/cassandra/repo
Commit: http://git-wip-us.apache.org/repos/asf/cassandra/commit/3bc585d5
Tree: http://git-wip-us.apache.org/repos/asf/cassandra/tree/3bc585d5
Diff: http://git-wip-us.apache.org/repos/asf/cassandra/diff/3bc585d5
Branch: refs/heads/trunk
Commit: 3bc585d5d0e01dd4f855994add67d3d120857eec
Parents: 0828e24 0d2f403
Author: Jonathan Ellis <jb...@apache.org>
Authored: Mon Aug 31 12:04:44 2015 -0500
Committer: Jonathan Ellis <jb...@apache.org>
Committed: Mon Aug 31 12:04:44 2015 -0500
----------------------------------------------------------------------
.../org/apache/cassandra/config/DatabaseDescriptor.java | 9 ++++++---
1 file changed, 6 insertions(+), 3 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/cassandra/blob/3bc585d5/src/java/org/apache/cassandra/config/DatabaseDescriptor.java
----------------------------------------------------------------------
[5/6] cassandra git commit: Merge branch 'cassandra-2.2' into
cassandra-3.0
Posted by jb...@apache.org.
Merge branch 'cassandra-2.2' into cassandra-3.0
Project: http://git-wip-us.apache.org/repos/asf/cassandra/repo
Commit: http://git-wip-us.apache.org/repos/asf/cassandra/commit/3bc585d5
Tree: http://git-wip-us.apache.org/repos/asf/cassandra/tree/3bc585d5
Diff: http://git-wip-us.apache.org/repos/asf/cassandra/diff/3bc585d5
Branch: refs/heads/cassandra-3.0
Commit: 3bc585d5d0e01dd4f855994add67d3d120857eec
Parents: 0828e24 0d2f403
Author: Jonathan Ellis <jb...@apache.org>
Authored: Mon Aug 31 12:04:44 2015 -0500
Committer: Jonathan Ellis <jb...@apache.org>
Committed: Mon Aug 31 12:04:44 2015 -0500
----------------------------------------------------------------------
.../org/apache/cassandra/config/DatabaseDescriptor.java | 9 ++++++---
1 file changed, 6 insertions(+), 3 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/cassandra/blob/3bc585d5/src/java/org/apache/cassandra/config/DatabaseDescriptor.java
----------------------------------------------------------------------
[6/6] cassandra git commit: Merge branch 'cassandra-3.0' into trunk
Posted by jb...@apache.org.
Merge branch 'cassandra-3.0' into trunk
Project: http://git-wip-us.apache.org/repos/asf/cassandra/repo
Commit: http://git-wip-us.apache.org/repos/asf/cassandra/commit/5d0c6ef7
Tree: http://git-wip-us.apache.org/repos/asf/cassandra/tree/5d0c6ef7
Diff: http://git-wip-us.apache.org/repos/asf/cassandra/diff/5d0c6ef7
Branch: refs/heads/trunk
Commit: 5d0c6ef77461b6b6b488be445c22062e0aa9b936
Parents: ae7afb7 3bc585d
Author: Jonathan Ellis <jb...@apache.org>
Authored: Mon Aug 31 12:04:51 2015 -0500
Committer: Jonathan Ellis <jb...@apache.org>
Committed: Mon Aug 31 12:04:51 2015 -0500
----------------------------------------------------------------------
.../org/apache/cassandra/config/DatabaseDescriptor.java | 9 ++++++---
1 file changed, 6 insertions(+), 3 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/cassandra/blob/5d0c6ef7/src/java/org/apache/cassandra/config/DatabaseDescriptor.java
----------------------------------------------------------------------
[3/6] cassandra git commit: Defer initialization of default
RoleManager until IAuthenticator is set reviewed by jbellis for
CASSANDRA-10209
Posted by jb...@apache.org.
Defer initialization of default RoleManager until IAuthenticator is set
reviewed by jbellis for CASSANDRA-10209
Project: http://git-wip-us.apache.org/repos/asf/cassandra/repo
Commit: http://git-wip-us.apache.org/repos/asf/cassandra/commit/0d2f4038
Tree: http://git-wip-us.apache.org/repos/asf/cassandra/tree/0d2f4038
Diff: http://git-wip-us.apache.org/repos/asf/cassandra/diff/0d2f4038
Branch: refs/heads/trunk
Commit: 0d2f4038fc2564c695d0d8c4b1ad523ec1afaceb
Parents: c870adc
Author: Sam Tunnicliffe <sa...@beobal.com>
Authored: Thu Aug 27 20:21:17 2015 +0100
Committer: Jonathan Ellis <jb...@apache.org>
Committed: Mon Aug 31 12:04:01 2015 -0500
----------------------------------------------------------------------
.../org/apache/cassandra/config/DatabaseDescriptor.java | 9 ++++++---
1 file changed, 6 insertions(+), 3 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/cassandra/blob/0d2f4038/src/java/org/apache/cassandra/config/DatabaseDescriptor.java
----------------------------------------------------------------------
diff --git a/src/java/org/apache/cassandra/config/DatabaseDescriptor.java b/src/java/org/apache/cassandra/config/DatabaseDescriptor.java
index b7e3eaa..9cf3959 100644
--- a/src/java/org/apache/cassandra/config/DatabaseDescriptor.java
+++ b/src/java/org/apache/cassandra/config/DatabaseDescriptor.java
@@ -87,7 +87,9 @@ public class DatabaseDescriptor
private static IAuthenticator authenticator = new AllowAllAuthenticator();
private static IAuthorizer authorizer = new AllowAllAuthorizer();
- private static IRoleManager roleManager = new CassandraRoleManager();
+ // Don't initialize the role manager until applying config. The options supported by CassandraRoleManager
+ // depend on the configured IAuthenticator, so defer creating it until that's been set.
+ private static IRoleManager roleManager;
private static IRequestScheduler requestScheduler;
private static RequestSchedulerId requestSchedulerId;
@@ -323,8 +325,9 @@ public class DatabaseDescriptor
if (authenticator instanceof AllowAllAuthenticator && !(authorizer instanceof AllowAllAuthorizer))
throw new ConfigurationException("AllowAllAuthenticator can't be used with " + conf.authorizer, false);
- if (conf.role_manager != null)
- roleManager = FBUtilities.newRoleManager(conf.role_manager);
+ roleManager = conf.role_manager == null
+ ? new CassandraRoleManager()
+ : FBUtilities.newRoleManager(conf.role_manager);
if (authenticator instanceof PasswordAuthenticator && !(roleManager instanceof CassandraRoleManager))
throw new ConfigurationException("CassandraRoleManager must be used with PasswordAuthenticator", false);
[2/6] cassandra git commit: Defer initialization of default
RoleManager until IAuthenticator is set reviewed by jbellis for
CASSANDRA-10209
Posted by jb...@apache.org.
Defer initialization of default RoleManager until IAuthenticator is set
reviewed by jbellis for CASSANDRA-10209
Project: http://git-wip-us.apache.org/repos/asf/cassandra/repo
Commit: http://git-wip-us.apache.org/repos/asf/cassandra/commit/0d2f4038
Tree: http://git-wip-us.apache.org/repos/asf/cassandra/tree/0d2f4038
Diff: http://git-wip-us.apache.org/repos/asf/cassandra/diff/0d2f4038
Branch: refs/heads/cassandra-3.0
Commit: 0d2f4038fc2564c695d0d8c4b1ad523ec1afaceb
Parents: c870adc
Author: Sam Tunnicliffe <sa...@beobal.com>
Authored: Thu Aug 27 20:21:17 2015 +0100
Committer: Jonathan Ellis <jb...@apache.org>
Committed: Mon Aug 31 12:04:01 2015 -0500
----------------------------------------------------------------------
.../org/apache/cassandra/config/DatabaseDescriptor.java | 9 ++++++---
1 file changed, 6 insertions(+), 3 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/cassandra/blob/0d2f4038/src/java/org/apache/cassandra/config/DatabaseDescriptor.java
----------------------------------------------------------------------
diff --git a/src/java/org/apache/cassandra/config/DatabaseDescriptor.java b/src/java/org/apache/cassandra/config/DatabaseDescriptor.java
index b7e3eaa..9cf3959 100644
--- a/src/java/org/apache/cassandra/config/DatabaseDescriptor.java
+++ b/src/java/org/apache/cassandra/config/DatabaseDescriptor.java
@@ -87,7 +87,9 @@ public class DatabaseDescriptor
private static IAuthenticator authenticator = new AllowAllAuthenticator();
private static IAuthorizer authorizer = new AllowAllAuthorizer();
- private static IRoleManager roleManager = new CassandraRoleManager();
+ // Don't initialize the role manager until applying config. The options supported by CassandraRoleManager
+ // depend on the configured IAuthenticator, so defer creating it until that's been set.
+ private static IRoleManager roleManager;
private static IRequestScheduler requestScheduler;
private static RequestSchedulerId requestSchedulerId;
@@ -323,8 +325,9 @@ public class DatabaseDescriptor
if (authenticator instanceof AllowAllAuthenticator && !(authorizer instanceof AllowAllAuthorizer))
throw new ConfigurationException("AllowAllAuthenticator can't be used with " + conf.authorizer, false);
- if (conf.role_manager != null)
- roleManager = FBUtilities.newRoleManager(conf.role_manager);
+ roleManager = conf.role_manager == null
+ ? new CassandraRoleManager()
+ : FBUtilities.newRoleManager(conf.role_manager);
if (authenticator instanceof PasswordAuthenticator && !(roleManager instanceof CassandraRoleManager))
throw new ConfigurationException("CassandraRoleManager must be used with PasswordAuthenticator", false);