Omega359 commented on code in PR #16970:
URL: https://github.com/apache/datafusion/pull/16970#discussion_r2245815418
##########
datafusion/core/src/execution/session_state.rs:
##########
@@ -738,6 +738,17 @@ impl SessionState {
self.config.options()
}
+ /// return the configuration options
+ pub fn config_options_arc(&self) -> Arc<ConfigOptions> {
+ self.config.options_arc()
+ }
+
+ /// Mark the start of the execution
Review Comment:
> Is it start of query execution? if so we might clarify the comments
This one came from @alamb's original POC. It's not actually starting the
execution, just setting the config options and the start execution time.
Perhaps mark_start_execution would be a more appropriate fn name.
--
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: github-unsubscr...@datafusion.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org
For additional commands, e-mail: github-h...@datafusion.apache.org
