adriangb commented on PR #16790: URL: https://github.com/apache/datafusion/pull/16790#issuecomment-3084406929
> I think a little more docs / context / comments are needed otherwise this is good to merge. @akoshchiy could you add some comments explaining what's going on for future reference, and documenting what the `__unnest_placeholder` is? I realize that's been around and is used for other things already but I think it's important to improve documentation and understanding as we go along so that next time we have an easier time fixing things. -- 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
