cassandra-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Joshua McKenzie (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (CASSANDRA-9847) Don't serialize CFMetaData in read responses
Date Fri, 24 Jul 2015 12:16:04 GMT

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

Joshua McKenzie commented on CASSANDRA-9847:
--------------------------------------------

+1 with a couple small nits
ReadResponse.java, extra whitespace snuck into DataResponse just after class decl
Brace on BigTableScanner.createIterator needs to be on a new line

> Don't serialize CFMetaData in read responses
> --------------------------------------------
>
>                 Key: CASSANDRA-9847
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-9847
>             Project: Cassandra
>          Issue Type: Sub-task
>          Components: Core
>            Reporter: Sylvain Lebresne
>            Assignee: Sylvain Lebresne
>             Fix For: 3.0 beta 1
>
>
> Our CFMetaData ids are 16 bytes long, which for small messages is a non trivial part
of the size (we're further currently unnecessarily serialize it with every partition). At
least for read response, we don't really need to serialize it at all since we always to which
query this is a response of.



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

Mime
View raw message