joellubi commented on PR #43890: URL: https://github.com/apache/arrow/pull/43890#issuecomment-2321867523
> The change seems reasonable to me, but we need to figure out what's wrong with the integration tests here before we can merge this The integration tests were failing because I included commits from #43834 which breaks the tests for a number of implementations. I confirmed that the Go scenarios in the workflow were passing on this PR, even though other languages are still failing with the changes to the tests. I removed the commits now and the integration testing workflow succeeded. -- 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]
