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

Johan Oskarsson commented on CASSANDRA-446:
-------------------------------------------

+1. The changes look good to me.
I assume the overhead of decorating the keys in the comparator in the 
StorageProxy won't be too bad if it's just a few entries being compared.

> Use DecoratedKey objects in Memtable, SSTableReader/Writer objects
> ------------------------------------------------------------------
>
>                 Key: CASSANDRA-446
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-446
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Core
>            Reporter: Jonathan Ellis
>             Fix For: 0.5
>
>         Attachments: CASSANDRA-446.patch, CASSANDRA-446.patch, 
> CASSANDRA-446.patch
>
>
> See CASSANDRA-420 for introduction of DecoratedKey.
> IMO we should move Memtable and SSTR objects to using DK objects, too.  This 
> will help Memtable flushes similar to how this helps BMt, and using DK 
> objects in SSTR will avoid a ton of potential confusion about whether a 
> String is a key or a decorated key.
> As a side benefit, moving the decoration operation out of memtable flush and 
> into the insert path will help make better use of multicore systems, since we 
> can have multiple write threads but only one core can flush at a time.  (See 
> CASSANDRA-445.)

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