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