saadtajwar commented on PR #23231: URL: https://github.com/apache/datafusion/pull/23231#issuecomment-4916076235
@gene-bordegaray pushed a commit to address your comment! Thank you all folks @gene-bordegaray & @gabotechs & @stuhood for taking the time to review! The `cargo fmt` CI test should pass now too 🙏 Quick bump on the above question, this probably just got lost in the shuffle of all of the comments & changes haha - @gene-bordegaray lmk if you have thoughts on this below! > The only open question I had was regarding the follow-ups for optimizations - just to clarify, would you like another GH issue opened to track changes like the one described https://github.com/apache/datafusion/pull/23231#discussion_r3514785766, beyond the existing https://github.com/apache/datafusion/issues/23230 issue that we've already opened? Or can https://github.com/apache/datafusion/issues/23230 encapsulate all of the follow-up work needed here? -- 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]
