adriangb commented on PR #19426:
URL: https://github.com/apache/datafusion/pull/19426#issuecomment-3677932266

   > Would that cached query then be passed through the optimizer again when 
executed (with the config reset to default), or would it be directly executed?
   > 
   > Because `now()`, `current_date()` and `current_time()` don't actually 
implement invoke, so if they aren't simplified they can't execute physically I 
believe:
   
   That's a good point, I had not noticed that. Personally I would have them 
implement `invoke!` (I'm guessing it's not too hard?). But I don't really need 
that: I would pass the non cached / materialized portion of the query through 
the optimizer again _right before it's executed_.


-- 
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]

Reply via email to