You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@arrow.apache.org by al...@apache.org on 2022/11/26 20:04:52 UTC

[arrow-datafusion] branch master updated: minor: improve optimizer logging and do not repeat rule name (#4345)

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

alamb pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/arrow-datafusion.git


The following commit(s) were added to refs/heads/master by this push:
     new 323fbb43a minor: improve optimizer logging and do not repeat rule name (#4345)
323fbb43a is described below

commit 323fbb43a3359d59bd12daf6d2b38102b4671d6f
Author: Andrew Lamb <an...@nerdnetworks.org>
AuthorDate: Sat Nov 26 15:04:45 2022 -0500

    minor: improve optimizer logging and do not repeat rule name (#4345)
---
 datafusion/optimizer/src/optimizer.rs | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/datafusion/optimizer/src/optimizer.rs b/datafusion/optimizer/src/optimizer.rs
index 2ac3fda23..340d246fe 100644
--- a/datafusion/optimizer/src/optimizer.rs
+++ b/datafusion/optimizer/src/optimizer.rs
@@ -239,7 +239,11 @@ impl Optimizer {
                     }
                     Ok(None) => {
                         observer(&new_plan, rule.as_ref());
-                        log_plan(rule.name(), &new_plan);
+                        debug!(
+                            "Plan unchanged by optimizer rule '{}' (pass {})",
+                            rule.name(),
+                            i
+                        );
                     }
                     Err(ref e) => {
                         if optimizer_config.skip_failing_rules {