[GitHub] [spark] cloud-fan commented on a change in pull request #27846: [SPARK-31079][SQL] Logging QueryExecutionMetering in RuleExecutor logger

[GitBox <gi...@apache.org>]

cloud-fan commented on a change in pull request #27846: [SPARK-31079][SQL] Logging QueryExecutionMetering in RuleExecutor logger
URL: https://github.com/apache/spark/pull/27846#discussion_r389520742
 
 

 ##########
 File path: sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/optimizer/OptimizerLoggingSuite.scala
 ##########
 @@ -49,12 +49,19 @@ class OptimizerLoggingSuite extends PlanTest {
       case event => Seq(
         "Applying Rule",
         "Result of Batch",
-        "has no effect").exists(event.getRenderedMessage().contains)
+        "has no effect",
+        "Metrics of Executed Rules").exists(event.getRenderedMessage().contains)
 
 Review comment:
   this only exists if `Result of Batch` exists?

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org