[
https://issues.apache.org/jira/browse/CASSANDRA-3223?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Yang Yang updated CASSANDRA-3223:
---------------------------------
Attachment: 0002-do-not-need-to-deep-copy-column-value-bytebuffer-now.patch
do not need to deep copy column value bytebuffer into row cache, now that we
already do this copy in JIRA 3179
> probably don't need to do full copy to row cache after un-mmap() change
> -----------------------------------------------------------------------
>
> Key: CASSANDRA-3223
> URL: https://issues.apache.org/jira/browse/CASSANDRA-3223
> Project: Cassandra
> Issue Type: Bug
> Components: Core
> Affects Versions: 1.0.0
> Reporter: Yang Yang
> Assignee: Yang Yang
> Priority: Minor
> Fix For: 1.0.0
>
> Attachments:
> 0002-do-not-need-to-deep-copy-column-value-bytebuffer-now.patch
>
>
> 3179 changes from directly using the bytebuffer from mmap(), to copying that
> buffer,
> CFS.cacheRow()
> https://github.com/apache/cassandra/blob/cassandra-1.0.0/src/java/org/apache/cassandra/db/ColumnFamilyStore.java
> line 1126
> says it makes a deep copy exactly to prevent issues from unmmap().
> maybe this deep copy is not needed now given 3179
> if so, maybe slightly better performance in both speed and memory
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira
