Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[ntuple] Properly support big-endian architectures #10383

Closed
jalopezg-git opened this issue Apr 12, 2022 · 1 comment · Fixed by #10402
Closed

[ntuple] Properly support big-endian architectures #10383

jalopezg-git opened this issue Apr 12, 2022 · 1 comment · Fixed by #10402

Comments

@jalopezg-git
Copy link
Collaborator

Explain what you would like to see improved

RNTuple metadata is serialized to a machine-independent representation. However, contents of a page are written using the native endianess of the architecture on which RNTuple runs. According to the RNTuple v1 specification, these should be laid out always in little-endian order.

Optional: share how it could be improved

If running on a big-endian machine, (un)packing of data pages for some data types should carry out a (16, 32, or 64-bit) byte swap, depending on the type.

@github-actions
Copy link

Hi @jalopezg-r00t,

It appears this issue is closed, but wasn't yet added to a project. Please add upcoming versions that will include the fix, or 'not applicable' otherwise.

Sincerely,
🤖

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant