address comments

pan3793 · pan3793 · commit 7d7e21ca4175 · 2025-09-12T10:24:07.000+08:00
diff --git a/externals/kyuubi-spark-sql-engine/src/test/scala/org/apache/spark/kyuubi/SparkSQLEngineDeregisterSuite.scala b/externals/kyuubi-spark-sql-engine/src/test/scala/org/apache/spark/kyuubi/SparkSQLEngineDeregisterSuite.scala
@@ -61,8 +61,8 @@ abstract class SparkSQLEngineDeregisterSuite
 class SparkSQLEngineDeregisterExceptionSuite extends SparkSQLEngineDeregisterSuite {
   override def withKyuubiConf: Map[String, String] = {
     super.withKyuubiConf ++
-      Map(ENGINE_DEREGISTER_EXCEPTION_CLASSES.key -> "org.apache.spark.SparkArithmeticException")
-
+      Map(ENGINE_DEREGISTER_EXCEPTION_CLASSES.key ->
+        classOf[SparkArithmeticException].getCanonicalName)
   }
 }
 
@@ -88,7 +88,8 @@ class SparkSQLEngineDeregisterExceptionTTLSuite
     super.withKyuubiConf ++
       zookeeperConf ++ Map(
         ANSI_ENABLED.key -> "true",
-        ENGINE_DEREGISTER_EXCEPTION_CLASSES.key -> "org.apache.spark.SparkArithmeticException",
+        ENGINE_DEREGISTER_EXCEPTION_CLASSES.key ->
+          classOf[SparkArithmeticException].getCanonicalName,
         ENGINE_DEREGISTER_JOB_MAX_FAILURES.key -> maxJobFailures.toString,
         ENGINE_DEREGISTER_EXCEPTION_TTL.key -> deregisterExceptionTTL.toString)
   }

Original file line number	Diff line number	Diff line change
`@@ -61,8 +61,8 @@ abstract class SparkSQLEngineDeregisterSuite`
`61`	`61`	`class SparkSQLEngineDeregisterExceptionSuite extends SparkSQLEngineDeregisterSuite {`
`62`	`62`	`override def withKyuubiConf: Map[String, String] = {`
`63`	`63`	`super.withKyuubiConf ++`
`64`		`- Map(ENGINE_DEREGISTER_EXCEPTION_CLASSES.key -> "org.apache.spark.SparkArithmeticException")`
`65`		`-`
	`64`	`+ Map(ENGINE_DEREGISTER_EXCEPTION_CLASSES.key ->`
	`65`	`+ classOf[SparkArithmeticException].getCanonicalName)`
`66`	`66`	`}`
`67`	`67`	`}`
`68`	`68`
`@@ -88,7 +88,8 @@ class SparkSQLEngineDeregisterExceptionTTLSuite`
`88`	`88`	`super.withKyuubiConf ++`
`89`	`89`	`zookeeperConf ++ Map(`
`90`	`90`	`ANSI_ENABLED.key -> "true",`
`91`		`- ENGINE_DEREGISTER_EXCEPTION_CLASSES.key -> "org.apache.spark.SparkArithmeticException",`
	`91`	`+ ENGINE_DEREGISTER_EXCEPTION_CLASSES.key ->`
	`92`	`+ classOf[SparkArithmeticException].getCanonicalName,`
`92`	`93`	`ENGINE_DEREGISTER_JOB_MAX_FAILURES.key -> maxJobFailures.toString,`
`93`	`94`	`ENGINE_DEREGISTER_EXCEPTION_TTL.key -> deregisterExceptionTTL.toString)`
`94`	`95`	`}`