wForget commented on code in PR #6112:
URL: https://github.com/apache/kyuubi/pull/6112#discussion_r1508741281


##########
externals/kyuubi-spark-sql-engine/src/main/scala/org/apache/spark/kyuubi/SQLOperationListener.scala:
##########
@@ -140,6 +149,14 @@ class SQLOperationListener(
     val stageInfo = stageCompleted.stageInfo
     val stageId = stageInfo.stageId
     val stageAttempt = SparkStageAttempt(stageInfo.stageId, 
stageInfo.attemptNumber())
+    val taskMetrics = stageInfo.taskMetrics

Review Comment:
   > But using the sameGroupId will miss broadcast operators. 
   
   This is not a reason not to determine whether stage belongs to current 
operation.



-- 
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]

Reply via email to