[
https://issues.apache.org/jira/browse/CASSANDRA-20190?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Dmitry Konstantinov updated CASSANDRA-20190:
--------------------------------------------
Description:
`NativeCell`, `NativeClustering` and `NativeDecoratedKey` use the above methods
from `MemoryUtil` to write and read data from native memory. As far as I can
see they are meant to write data in big endian. They do not (they always
correct to little endian).
Moreover, they disagree with their `ByByte` versions on big-endian machines
(which is only likely an issue on aligned-access architectures (x86 and arm
should be fine)).
The same is true for the methods in `Memory`, used by compression metadata as
well as index summaries.
We need to verify that this does not cause any problems, and to change the
methods to behave as expected and document the behaviour by explicitly using
`ByteOrder.LITTLE_ENDIAN` for any data that may have been persisted on disk
with the wrong endianness.
The current MemoryUtil behaviour:
||Native
order||MemoryUtil.putX||MemoryUtil.putXByByte||MemoryUtil.getX||MemoryUtil.getXByByte||
|BE|LE|BE|LE|BE|
|LE|LE|LE|LE|LE|
was:
`NativeCell`, `NativeClustering` and `NativeDecoratedKey` use the above methods
from `MemoryUtil` to write and read data from native memory. As far as I can
see they are meant to write data in big endian. They do not (they always
correct to little endian).
Moreover, they disagree with their `ByByte` versions on big-endian machines
(which is only likely an issue on aligned-access architectures (x86 and arm
should be fine)).
The same is true for the methods in `Memory`, used by compression metadata as
well as index summaries.
We need to verify that this does not cause any problems, and to change the
methods to behave as expected and document the behaviour by explicitly using
`ByteOrder.LITTLE_ENDIAN` for any data that may have been persisted on disk
with the wrong endianness.
> MemoryUtil.setInt/getInt and similar use the wrong endianness
> -------------------------------------------------------------
>
> Key: CASSANDRA-20190
> URL: https://issues.apache.org/jira/browse/CASSANDRA-20190
> Project: Apache Cassandra
> Issue Type: Bug
> Components: Local/Other
> Reporter: Branimir Lambov
> Priority: Normal
>
> `NativeCell`, `NativeClustering` and `NativeDecoratedKey` use the above
> methods from `MemoryUtil` to write and read data from native memory. As far
> as I can see they are meant to write data in big endian. They do not (they
> always correct to little endian).
> Moreover, they disagree with their `ByByte` versions on big-endian machines
> (which is only likely an issue on aligned-access architectures (x86 and arm
> should be fine)).
> The same is true for the methods in `Memory`, used by compression metadata as
> well as index summaries.
> We need to verify that this does not cause any problems, and to change the
> methods to behave as expected and document the behaviour by explicitly using
> `ByteOrder.LITTLE_ENDIAN` for any data that may have been persisted on disk
> with the wrong endianness.
> The current MemoryUtil behaviour:
> ||Native
> order||MemoryUtil.putX||MemoryUtil.putXByByte||MemoryUtil.getX||MemoryUtil.getXByByte||
> |BE|LE|BE|LE|BE|
> |LE|LE|LE|LE|LE|
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
