zanmato1984 commented on PR #45336: URL: https://github.com/apache/arrow/pull/45336#issuecomment-2609697645
Hi @pitrou . Hopefully the CI should be fixed now. And the test has been enhanced to the extent that can also discover the overflow in key columns (the position of the key in row table depends on the hashing algorithm so it took me a while to tune a matching key value that will locate on the higher address in the row table - note the `289339070` in the test). I tested it by reverting several places of the fix in my local and the overflow did happen. I'm addressing your comments now. Will update soon. -- 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]
