alamb commented on code in PR #9537:
URL: https://github.com/apache/arrow-datafusion/pull/9537#discussion_r1523460945
##########
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 think moving the tests to the core (what this PR does) is probably the
best solution for now, as it correctly reflects the dependencies (the optimizer
tests are testing behavior of functions that are not available to optimizer)
--
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]
