You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@datafusion.apache.org by al...@apache.org on 2025/07/08 14:55:35 UTC
(datafusion) branch main updated: Improve display output for FFI execution plans (#16713)
This is an automated email from the ASF dual-hosted git repository.
alamb pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/datafusion.git
The following commit(s) were added to refs/heads/main by this push:
new e45de7f3a5 Improve display output for FFI execution plans (#16713)
e45de7f3a5 is described below
commit e45de7f3a554c51061288731fe2dcaa996c94299
Author: Tim Saucer <ti...@gmail.com>
AuthorDate: Tue Jul 8 10:55:30 2025 -0400
Improve display output for FFI execution plans (#16713)
---
datafusion/ffi/src/execution_plan.rs | 8 ++++++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/datafusion/ffi/src/execution_plan.rs b/datafusion/ffi/src/execution_plan.rs
index 14a0908c47..70c957d8c3 100644
--- a/datafusion/ffi/src/execution_plan.rs
+++ b/datafusion/ffi/src/execution_plan.rs
@@ -205,7 +205,8 @@ impl DisplayAs for ForeignExecutionPlan {
DisplayFormatType::Default | DisplayFormatType::Verbose => {
write!(
f,
- "FFI_ExecutionPlan(number_of_children={})",
+ "FFI_ExecutionPlan: {}, number_of_children={}",
+ self.name,
self.children.len(),
)
}
@@ -390,7 +391,10 @@ mod tests {
);
let buf = display.one_line().to_string();
- assert_eq!(buf.trim(), "FFI_ExecutionPlan(number_of_children=0)");
+ assert_eq!(
+ buf.trim(),
+ "FFI_ExecutionPlan: empty-exec, number_of_children=0"
+ );
Ok(())
}
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@datafusion.apache.org
For additional commands, e-mail: commits-help@datafusion.apache.org