mihaibudiu commented on PR #5086: URL: https://github.com/apache/calcite/pull/5086#issuecomment-4964123606
Interestingly, Claude actually found a (related) bug in the original optimization, which forgot to preserve fields NOT moved into `requiredColumns`. Several test cases cover these failures. I haven't filed a separate issue since there is common code between this "improvement" and the bug fix itself. As a consequence of this fix several plans have changed, but they are equivalent and always better. -- 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]
