arminazimi commented on PR #7004: URL: https://github.com/apache/datafusion-comet/pull/7004#issuecomment-6268152807
Thank you, @andygrove, for the thoughtful review and clear guidance! I really appreciate your time and the kind words about the LargeUtf8 test. I've rebased onto main, preserved the validity buffers from #6953 with the expressions you suggested, and removed the redundant array/array test. The updated commit is 0f9be90. Rust checks have not been rerun in this environment; CI is pending maintainer approval. Thanks again for helping me get this into shape! -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
