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

Ted Zlatanov commented on CASSANDRA-547:
----------------------------------------

I left the libthrift upgrade out of my patches, but it's probably a good idea 
to do it as some point.

There are two patches, the auth one is a superset of the thrift one.

ThriftGlue has one (IMHO) nice feature: it lets us create ColumnOrSuperColumn 
objects with the explicit inclination to contain a Column or a SuperColumn.  
The old way was less readable.

I remove the style changes for the useless else clauses.

I made ConsistencyLevel go all the way down the stack and changed the variable 
name to consistencyLevel while I was there, to be consistent (ha ha) with the 
other parameter names.  There is one place where the int value is used to check 
enum superiority, but overall it looks and works better I think.

Let me know if all of this looks good.  It's a real PITA to keep patching so I 
hope we're close to done.

> authentication and authorization functions (stage 1)
> ----------------------------------------------------
>
>                 Key: CASSANDRA-547
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-547
>             Project: Cassandra
>          Issue Type: New Feature
>          Components: Core
>    Affects Versions: 0.5
>            Reporter: Ted Zlatanov
>            Assignee: Ted Zlatanov
>         Attachments: CASSANDRA-547-auth.patch, CASSANDRA-547-thrift.patch, 
> libthrift-r893324.jar
>
>
> Add functions to login() and setKeyspace().  Check against 
> authentication+authorization backends whether those calls should succeed or 
> fail.
> Create file-based and LDAP backends to implement authentication and 
> authorization.
> Create AllowAll backend to allow all access.
> Add configuration stanza to specify the auth backend.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to