xuzifu666 commented on code in PR #43570:
URL: https://github.com/apache/spark/pull/43570#discussion_r1375228907
##########
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/QueryPlanningTracker.scala:
##########
@@ -134,7 +134,7 @@ class QueryPlanningTracker(
* and the recorded end time will be the end time of the last call.
*/
def measurePhase[T](phase: String)(f: => T): T = {
- val startTime = System.currentTimeMillis()
+ val startTime = System.currentTimeMillis
Review Comment:
no error or warning，just a code refactor for it，starttime with() and endtime
not，is a little strange，if not necessary please close it，thanks @LuciferYang
--
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]
