zanmato1984 commented on PR #39685: URL: https://github.com/apache/arrow/pull/39685#issuecomment-2044425913
> 64-bit here should be fine. This is the defect I was thinking of originally: #34474 > > As long as we are only changing `uint32_t` to `int64_t` and all the tests pass then I don't think we need to worry about breaking large memory support. Hi @westonpace , sorry about coming back to this late. In the quoted comment, are you suggesting keeping the row-offsets/byte-offsets unsigned and only changing the number of rows/bytes to signed 64-bit? Or changing both the offsets and numbers to signed 64-bit? Please help to confirm and I'll keep working on this. Thank you. -- 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]
