wgtmac commented on PR #34416: URL: https://github.com/apache/arrow/pull/34416#issuecomment-1454753930
> Does your test cover the read path? Also, do you want a test with a dense union? Yes, I have added a roundtrip test for sparse union type. The read path of dense union will never happen because we always convert orc union to arrow sparse union type. Let me add a test case for batch conversion roundtrip of dense union to make it coverred. -- 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]
