dongjoon-hyun commented on code in PR #52678:
URL: https://github.com/apache/spark/pull/52678#discussion_r2449594307


##########
core/src/main/scala/org/apache/spark/util/Utils.scala:
##########
@@ -3138,7 +3138,14 @@ private[spark] object Utils
   /**
    * Return whether we are using G1GC or not
    */
-  lazy val isG1GC: Boolean = {
+  lazy val isG1GC: Boolean = checkUseGC("UseG1GC")
+
+  /**
+   * Return whether we are using ZGC or not
+   */
+  lazy val isZGC: Boolean = checkUseGC("UseZGC")
+
+  def checkUseGC(useGCObjectStr: String): Boolean = {

Review Comment:
   Hi, @wankunde . Thank you for making a PR. Could you spin off this 
contribution? This sounds like a good idea to add a general utility function to 
detect GCs. We can proceed this one first.
   
   As @pan3793 mentioned, if you can generalize more to detect other GCs, it 
would be a perfect contribution.



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