This is an automated email from the ASF dual-hosted git repository.

zhuqi pushed a commit to branch branch-50
in repository https://gitbox.apache.org/repos/asf/datafusion.git


The following commit(s) were added to refs/heads/branch-50 by this push:
     new 513ba7a59c Fix potential overflow when we print verbose physical plan 
(#17798) (#17804)
513ba7a59c is described below

commit 513ba7a59cf261e06d8d68e3372577f892bf5ead
Author: Qi Zhu <[email protected]>
AuthorDate: Sat Sep 27 11:20:39 2025 +0800

    Fix potential overflow when we print verbose physical plan (#17798) (#17804)
---
 datafusion/core/src/physical_planner.rs | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

diff --git a/datafusion/core/src/physical_planner.rs 
b/datafusion/core/src/physical_planner.rs
index 6618d9495d..85a3036fe5 100644
--- a/datafusion/core/src/physical_planner.rs
+++ b/datafusion/core/src/physical_planner.rs
@@ -2093,7 +2093,15 @@ impl DefaultPhysicalPlanner {
             "Optimized physical plan:\n{}\n",
             displayable(new_plan.as_ref()).indent(false)
         );
-        debug!("Detailed optimized physical plan:\n{new_plan:?}");
+
+        // Don't print new_plan directly, as that may overflow the stack.
+        // For example:
+        // thread 'tokio-runtime-worker' has overflowed its stack
+        // fatal runtime error: stack overflow, aborting
+        debug!(
+            "Detailed optimized physical plan:\n{}\n",
+            displayable(new_plan.as_ref()).indent(true)
+        );
         Ok(new_plan)
     }
 


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to