danepitkin commented on PR #38016: URL: https://github.com/apache/arrow/pull/38016#issuecomment-1809267796
> One tidbit passed on from the Trino team was that once we have FFI then we'd also want to figure out how to move past any uses of ByteBuffer in the rest of the code to the FFI classes, as (reportedly) some mistakes were made with ByteBuffer which means it isn't very friendly to inlining and the new stuff is much much faster once inlining has time to kick in. Thank you! I envision this refactor, plus supporting `long` memory addresses, as follow up tasks. I can file those if this PR merges without those features. -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
