mathyingzhou commented on pull request #8648: URL: https://github.com/apache/arrow/pull/8648#issuecomment-773880147
I will make some further changes to the code to address Micah Kornfield's concerns but pretty much that's it. Further simplification of the tests require serious expansion of arrow/testing/random and arrow/compute/cast to include nested types (and in the case of the latter at least the fixed_size_binary -> binary cast) or it will remain very ugly. If possible I can work on such features in at least 2 separate PRs. ---------------------------------------------------------------- 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