sdf-jkl commented on PR #20059: URL: https://github.com/apache/datafusion/pull/20059#issuecomment-3830200400
Hi @devanshu0987, I didn't have a chance to take a close look at everything, but I saw you're using unit tests instead of slt tests. I'm working on a similar issue on `date_part` function #19733, and there I only added new slt tests and no unit tests. A few of those use `EXPLAIN` to make sure the original function no longer appears in the logical and physical plans. -- 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]
