alamb commented on issue #27429: URL: https://github.com/apache/datafusion/issues/27429#issuecomment-6878091486
I suspect @NGA-TRAN would also be interested in this Another thing I think that might be helpful is to update the eqivalence property tests to use a style that is easier to read For example, it is pretty hard to see what this test does: https://github.com/apache/datafusion/blob/4ca40d9107fa760704e7717fe0fa542bd5ed6047/datafusion/physical-expr/src/equivalence/class.rs#L1591-L1639 Maybe we could rewrite it in transcript style: https://andrew.nerdnetworks.org/blog/2026/11/13/transcript-testing/ (sorry for he self promotion, but a large part of why I wrote that post was to have an easy reference / example) -- 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]
