You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cassandra.apache.org by "Jonathan Ellis (JIRA)" <ji...@apache.org> on 2009/11/30 17:01:20 UTC

[jira] Issue Comment Edited: (CASSANDRA-569) Add Thrift API to set per-connection settings

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

Jonathan Ellis edited comment on CASSANDRA-569 at 11/30/09 3:59 PM:
--------------------------------------------------------------------

Thanks for the patch!  A question: 

Why do we need resetLocalSettings in getProcessor?  Since we are using threaded server, shouldn't the threadlocalness handle that for us when a new client connects?

Also, since we are moving towards using DatabaseDescriptor for fat clients as well as servers (see CASSANDRA-535), StorageProxy is a better place for the threadlocal.

      was (Author: jbellis):
    Why do we need resetLocalSettings in getProcessor?  Since we are using threaded server, shouldn't the threadlocalness handle that for us when a new client connects?

Since we are moving towards using DatabaseDescriptor for fat clients as well as servers (see CASSANDRA-535), StorageProxy is a better place for the threadlocal.
  
> Add Thrift API to set per-connection settings
> ---------------------------------------------
>
>                 Key: CASSANDRA-569
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-569
>             Project: Cassandra
>          Issue Type: New Feature
>          Components: Core
>            Reporter: Paul Querna
>         Attachments: CASSANDRA-569.patch
>
>
> For thrift API, let you set configuration options per-connection.  These would be stored thread-local on the server, and when a thrift connection dies, they would be reset.
> An example use case is setting the timeout for requests on a per-connection basis.

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