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

wenchen pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/spark.git


The following commit(s) were added to refs/heads/master by this push:
     new 1e717fe2b556 [SPARK-51776][SQL] Fix logging in single-pass Analyzer
1e717fe2b556 is described below

commit 1e717fe2b5569ac5f62ffd8a32221bd28bfa9cb6
Author: Vladimir Golubev <vladimir.golu...@databricks.com>
AuthorDate: Mon Apr 14 10:49:38 2025 +0800

    [SPARK-51776][SQL] Fix logging in single-pass Analyzer
    
    ### What changes were proposed in this pull request?
    
    Fix logging in single-pass Analyzer.
    
    ### Why are the changes needed?
    
    `lookupMetadataAndResolve` logging is out of place.
    
    ### Does this PR introduce _any_ user-facing change?
    
    No.
    
    ### How was this patch tested?
    
    Existing tests.
    
    ### Was this patch authored or co-authored using generative AI tooling?
    
    No.
    
    Closes #50565 from 
vladimirg-db/vladimir-golubev_data/single-pass-analyzer/fix-logging.
    
    Authored-by: Vladimir Golubev <vladimir.golu...@databricks.com>
    Signed-off-by: Wenchen Fan <wenc...@databricks.com>
---
 .../org/apache/spark/sql/catalyst/analysis/resolver/Resolver.scala    | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git 
a/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/resolver/Resolver.scala
 
b/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/resolver/Resolver.scala
index d82025b29509..3103e64885f7 100644
--- 
a/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/resolver/Resolver.scala
+++ 
b/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/resolver/Resolver.scala
@@ -145,10 +145,10 @@ class Resolver(
         relationMetadataProvider
     }
 
-    planLogger.logPlanResolutionEvent(planAfterSubstitution, "Main resolution")
-
     relationMetadataProvider.resolve(planAfterSubstitution)
 
+    planLogger.logPlanResolutionEvent(planAfterSubstitution, "Main resolution")
+
     planAfterSubstitution.setTagValue(Resolver.TOP_LEVEL_OPERATOR, ())
 
     resolve(planAfterSubstitution)


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@spark.apache.org
For additional commands, e-mail: commits-h...@spark.apache.org

Reply via email to