kosiew commented on code in PR #17021:
URL: https://github.com/apache/datafusion/pull/17021#discussion_r2297533222
##########
datafusion-cli/src/cli_context.rs:
##########
@@ -52,6 +52,19 @@ pub trait CliSessionContext {
&self,
plan: LogicalPlan,
) -> Result<DataFrame, DataFusionError>;
+
+ /// Return true if memory profiling is enabled.
+ fn memory_profiling(&self) -> bool {
Review Comment:
That would require print_options to be mut because we can
toggle memory_profiling
It was moved out of print_options after a comment that print_options should
not be mut for memory profiling.
--
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
