Omega359 commented on code in PR #9537:
URL: https://github.com/apache/arrow-datafusion/pull/9537#discussion_r1521370910
##########
datafusion/core/tests/optimizer_integration.rs:
##########
@@ -342,7 +343,12 @@ fn test_sql(sql: &str) -> Result<LogicalPlan> {
let statement = &ast[0];
// create a logical query plan
- let context_provider = MyContextProvider::default();
+ let now_udf = datetime::functions()
Review Comment:
I'll take a look. 6/20 tests use functions in the test (concat, sum,
concat_ws, now(), avg). I can move just those over for sure. I wish this test
dependency issue wasn't such an problem in Rust - most of the problems I've
seen migrating the functions have been test related.
--
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]
