alamb commented on PR #14273: URL: https://github.com/apache/datafusion/pull/14273#issuecomment-2614439327
> We definitely shouldn't merge anything that might introduce regressions without going through the very reasonable process you suggested. It didn't seem to me this would cause regressions, but if that's the case, we should wait and understand better. 👍 > This week, I will start off by thinking about what we can do about the correctness issue the other PR is focusing on -- without regressions. It'd be great if we can find a neat fix for the issue that doesn't have the performance implication instead of just reverting for now. I have the feeling that there is a way, but we'll see. I think @jayzhan211 has a good idea here: - https://github.com/apache/datafusion/pull/14223#issuecomment-2613874994 -- 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
