cassandra-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Robert Stupp (JIRA)" <>
Subject [jira] [Commented] (CASSANDRA-12381) Verify use of ByteBuffer.hasArray() in whole code base
Date Thu, 04 Aug 2016 15:55:20 GMT


Robert Stupp commented on CASSANDRA-12381:

Strictly, we have some misuses (assuming {{!hasArray()}} == direct buffer) for example in
It is not an issue now (as it is exchangeable as of the current OpenJDK/Oracle implementation)
- but should be fixe IMO. WDYT?

> Verify use of ByteBuffer.hasArray() in whole code base
> ------------------------------------------------------
>                 Key: CASSANDRA-12381
>                 URL:
>             Project: Cassandra
>          Issue Type: Bug
>            Reporter: Robert Stupp
>            Assignee: Robert Stupp
>            Priority: Minor
>             Fix For: 3.0.x
> As [noted here|],
a direct {{ByteBuffer}} can have a backing array.
> If a direct {{ByteBuffer}} has a backing array, we should not make any assumption on
how it is actually used.
> This ticket is about to check whether uses of {{ByteBuffer.hasArray()}} need to be replaced
with {{ByteBuffer.isDirect()}}.
> (With CASSANDRA-11870 however, there is no way that such a direct {{ByteBuffer}} has
a backing array.)

This message was sent by Atlassian JIRA

View raw message