cassandra-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Benedict (JIRA)" <j...@apache.org>
Subject [jira] [Comment Edited] (CASSANDRA-6689) Partially Off Heap Memtables
Date Thu, 20 Mar 2014 23:13:44 GMT

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

Benedict edited comment on CASSANDRA-6689 at 3/20/14 11:13 PM:
---------------------------------------------------------------

See my comment explaining [here|https://issues.apache.org/jira/browse/CASSANDRA-6694?focusedCommentId=13904708&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-13904708].
It sets up for the off-heap changes, I suppose we could delay the refactor, but it's just
more work since they're all targeted for 2.1, and it's not harmful to make it part of this
refactor.

bq. One more nit thing, in DecoratedKey.java there is no need to mark token() and key() explicitly
"abstract", also token() is already defined in RingPosition so no need to declare it in DecoratedKey.

The latter is an artefact of merging the getToken() and token() into the same method, and
I agree. Although there's no harm in either, happy to change them.


was (Author: benedict):
See my comment explaining [here|https://issues.apache.org/jira/browse/CASSANDRA-6694?focusedCommentId=13904708&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-13904708]

bq. One more nit thing, in DecoratedKey.java there is no need to mark token() and key() explicitly
"abstract", also token() is already defined in RingPosition so no need to declare it in DecoratedKey.

The latter is an artefact of merging the getToken() and token() into the same method, and
I agree. Although there's no harm in either, happy to change them.

> Partially Off Heap Memtables
> ----------------------------
>
>                 Key: CASSANDRA-6689
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-6689
>             Project: Cassandra
>          Issue Type: New Feature
>          Components: Core
>            Reporter: Benedict
>            Assignee: Benedict
>              Labels: performance
>             Fix For: 2.1 beta2
>
>         Attachments: CASSANDRA-6689-small-changes.patch
>
>
> Move the contents of ByteBuffers off-heap for records written to a memtable.
> (See comments for details)



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Mime
View raw message