dohongdayi commented on a change in pull request #33491:
URL: https://github.com/apache/spark/pull/33491#discussion_r741858933



##########
File path: 
core/src/main/scala/org/apache/spark/internal/config/ConfigBuilder.scala
##########
@@ -279,3 +286,8 @@ private[spark] case class ConfigBuilder(key: String) {
     }
   }
 }
+
+object EffectTiming extends Enumeration {
+  type EffectTiming = Value
+  val LAUNCHING_DRIVER, DEPLOYING_APPLICATION, RUNTIME, DEPLOYING_CLUSTER, 
TESTING, UNKNOWN = Value

Review comment:
       Would you like to add comments to describe each of them?
   For me, I'm a little confuse on LAUNCHING_DRIVER, DEPLOYING_APPLICATION and 
DEPLOYING_CLUSTER




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