cassandra-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Sylvain Lebresne (JIRA)" <>
Subject [jira] [Commented] (CASSANDRA-10761) Possible regression of CASSANDRA-9201
Date Wed, 02 Dec 2015 15:13:11 GMT


Sylvain Lebresne commented on CASSANDRA-10761:

A few small remarks (that really only apply to 3.0):
# While this is not a problem of this patch, I don't think {{Schema.load}} should be called
first as it is. It populates {{cfIdMap}} which intrinsically points to the {{keyspaces}} map,
so it doesn't make sense to do that before we've even added the new table to {{keyspaces}}
(which should itself come after initialization).
# Not a fan of doing the working in the closure passed to {{Schema.update}}: that closure
is fundamentally a tranformation of its {{KeyspaceMetadata}} argument and doing side effect
in it should be avoided as it adds nothing. I'll note that this is also not new to this patch.
# We should modify {{Schema.addView}} in a similar way.

To clarify what I have in mind, I've pushed changes for those remarks [here|].
With those changes I'm +1 on the rest, so if you agree on said changes, would you mind pulling
them in your branches and checking CI before committing?

> Possible regression of CASSANDRA-9201
> -------------------------------------
>                 Key: CASSANDRA-10761
>                 URL:
>             Project: Cassandra
>          Issue Type: Sub-task
>            Reporter: Philip Thompson
>            Assignee: Sam Tunnicliffe
>             Fix For: 3.0.1, 3.1, 2.2.x
>         Attachments: 10761-logs.tar.gz
> Some dtests like {{consistency_test.TestAccuracy.test_network_topology_strategy_each_quorum_counters}}
are failing with the follow auth related assertion exception
> {code}
> [node6 ERROR] java.lang.AssertionError: org.apache.cassandra.exceptions.InvalidRequestException:
unconfigured table roles
> 	at org.apache.cassandra.auth.CassandraRoleManager.prepare(
> 	at org.apache.cassandra.auth.CassandraRoleManager.setup(
> 	at org.apache.cassandra.service.StorageService.doAuthSetup(
> 	at org.apache.cassandra.service.StorageService.joinTokenRing(
> 	at org.apache.cassandra.service.StorageService.initServer(
> 	at org.apache.cassandra.service.StorageService.initServer(
> 	at org.apache.cassandra.service.CassandraDaemon.setup(
> 	at org.apache.cassandra.service.CassandraDaemon.activate(
> 	at org.apache.cassandra.service.CassandraDaemon.main(
> Caused by: org.apache.cassandra.exceptions.InvalidRequestException: unconfigured table
> 	at org.apache.cassandra.thrift.ThriftValidation.validateColumnFamily(
> 	at org.apache.cassandra.cql3.statements.SelectStatement$RawStatement.prepare(
> 	at org.apache.cassandra.cql3.statements.SelectStatement$RawStatement.prepare(
> 	at org.apache.cassandra.auth.CassandraRoleManager.prepare(
> 	... 8 more
> {code}
> This looks very similar to CASSANDRA-9201.

This message was sent by Atlassian JIRA

View raw message