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

Eric Evans commented on CASSANDRA-714:
--------------------------------------

bq. I didn't want to make too many changes within a single patch. Should each 
be addressed separately and all patches merged together, or what is the best 
procedure?

I think it's OK to leave the cli broken for a bit, so we can move forward and 
address CASSANDRA-859 after. The func tests (test_thrift_server.py) and 
hadoop.ColumnFamilyRecordReader though should go in with this, but addressing 
them in separate patches will make it easier to review.

Thanks Todd

> remove keyspace argument from thrift methods
> --------------------------------------------
>
>                 Key: CASSANDRA-714
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-714
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Core
>    Affects Versions: 0.6
>            Reporter: Eric Evans
>             Fix For: 0.7
>
>         Attachments: CASSANDRA-714.patch
>
>
> With the completion of CASSANDRA-547, the keyspace can be specified as part 
> of the login and stored in a thread-local. This will allow us to remove the 
> wasteful and redundant mandatory keyspace argument on all of the other rpc 
> methods.

-- 
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