adriangb commented on PR #20229: URL: https://github.com/apache/datafusion/pull/20229#issuecomment-3909031265
@Sahitya0805 i’m sorry if I was not clear. I was just suggesting an idea that I think needs buy in from other maintainers before we go forward with it. It makes the PR much larger so it has to show a clear advantage over the simple less than 100 line version. I’m sorry if this was resulted in you spending a lot of time on it. I suggest you revert this version for now, especially because it is bound to accumulate merge conflicts, and we put up the solution that Andrew suggested in https://github.com/apache/datafusion/pull/20229#discussion_r2788465935. Thanks. -- 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]
