cassandra-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Sylvain Lebresne (JIRA)" <>
Subject [jira] [Updated] (CASSANDRA-14057) The size of a byte is not 2
Date Fri, 17 Nov 2017 14:49:01 GMT


Sylvain Lebresne updated CASSANDRA-14057:
    Status: Patch Available  (was: Open)

> The size of a byte is not 2
> ---------------------------
>                 Key: CASSANDRA-14057
>                 URL:
>             Project: Cassandra
>          Issue Type: Bug
>          Components: Streaming and Messaging
>            Reporter: Sylvain Lebresne
>            Assignee: Sylvain Lebresne
>            Priority: Trivial
>             Fix For: 3.0.16, 3.11.2
> {{DataLimits}} serializer uses {{TypeSizes.sizeof((byte)limits.kind().ordinal())}} in
it's {{serializedSize}} method, but {{TypeSizes}} does not (on 3.0/3.11) have a {{sizeof(byte)}}
override, so {{sizeof(short)}} is picked and it returns 2, which is wrong.
> This is actually fixed on trunk, [~jasobrown] committed the fix as part of CASSANDRA-8457,
but it wasn't committed in 3.0/3.11 and that still feel dodgy to leave as is.
> To be clear, I don't think it's a problem as of now: it does break the size computed
for read commands, and thus the payload size of such message, but said payload size is not
use (for read requests that is).
> Still, not reason to leave something wrong like this and risk a future bug when the fix
is trivial.

This message was sent by Atlassian JIRA

To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message