JasonLi-cn commented on PR #11469: URL: https://github.com/apache/datafusion/pull/11469#issuecomment-2229933369
> Thank you @JasonLi-cn -- I think this PR looks good to me. Thank you for the contribution > > I think adding some comments about what `try_process_group_by_unnest` is doing and why would help improve this code but I don't think it is required prior to merge > > c @jonahgao Thank you @alamb for your review. I have completed the code modification according to your modification suggestions. -- 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: github-unsubscr...@datafusion.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org For additional commands, e-mail: github-h...@datafusion.apache.org
