mihailotim-db commented on code in PR #50366:
URL: https://github.com/apache/spark/pull/50366#discussion_r2011474744
##########
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/rules/QueryExecutionMetering.scala:
##########
@@ -108,6 +108,10 @@ case class QueryExecutionMetering() {
}
}
+object QueryExecutionMetering {
+ val queryExecutionMeter = QueryExecutionMetering()
Review Comment:
I think we should leave it as a case class in case we want to use separate
objects, similar to how we do with `QueryPlanningTracker`. I think it just
happens that we don't use this at the moment, but could be useful especially in
tests. Up to you, but IMO not necessary
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
