cassandra-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Ariel Weisberg (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (CASSANDRA-10189) Unify read/writeUTF code paths
Date Thu, 27 Aug 2015 17:02:46 GMT

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

Ariel Weisberg commented on CASSANDRA-10189:
--------------------------------------------

bq. I was referring to the zero-copy allocation for the large strings. I should have made
clearer that I completely agree for small strings.
Ah that makes sense. Yeah the argument for optimizing small strings works against messy code
for large ones.

> Unify read/writeUTF code paths
> ------------------------------
>
>                 Key: CASSANDRA-10189
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-10189
>             Project: Cassandra
>          Issue Type: Improvement
>            Reporter: Robert Stupp
>            Assignee: Robert Stupp
>
> (Follow-up to CASSANDRA-9738 and CASSANDRA-8670)
> CASSANDRA-9738 requires {{writeUTF}} functionality that has been improved in CASSANDRA-8670
plus {{readUTF}} functionality. But we need slightly different signatures - one taking {{DataInput}}/{{DataOutput}}
and one taking {{ByteBuffer}}.
> We can combine both code paths and benefit from a shared, thread-local byte buffer.
> Slightly different implementations are needed for array backed and direct BBs (as we
can directly access the backing array bypassing the direct BB's boundary checks).
> (Part of this has already been done for CASSANDRA-9738 in {{OHCKeyCache.SerializationUtil}})



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message