kiszk commented on issue #6981: URL: https://github.com/apache/arrow/pull/6981#issuecomment-616507292
At first, this PR address only test cases. This PR does not address the routines (like `column_writer.h`) yet. To update test cases can clarify the specification and implementation like this. In this change, int96, which is a pair of uint64 and uint32, is stored in memory as follows: ``` For little-endian Int96 Int96_min = {{1024, 2048, 4096}}; 00 04 00 00 00 08 00 00 00 10 00 00 ~~~~~~~~~~~~~~~~~~~~~~~ ~~~~~~~~~~~ uint64 (LE) uint32 (LE) For big-endian, Int96 Int96_min = {{2048, 1024, 4096}}; 00 00 08 00 00 00 04 00 00 00 10 00 ~~~~~~~~~~~~~~~~~~~~~~~ ~~~~~~~~~~~ uint64 (BE) uint32 (BE) ``` ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org