akurmustafa commented on issue #12700: URL: https://github.com/apache/datafusion/issues/12700#issuecomment-2412019071
> I have worked through [this comment](https://github.com/apache/datafusion/issues/12700#issuecomment-2389022925) in detail to determine whether there is an actual bug in the current implementation. Initially, my thoughts were similar to @akurmustafa's intuition, and I draft a few designs. However, I can now safely say that the current implementation is correct, and the tests are valid for both the `add_new_orderings()` and `satisfy()` APIs. Below are my brief findings: Thanks @berkaysynnada for this finding. Nice catch. I hope, I didn't waste your time by misdirecting. -- 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]
