[kyuubi] branch debug updated: addr comments

[yao]

This is an automated email from the ASF dual-hosted git repository.

yao pushed a commit to branch debug
in repository https://gitbox.apache.org/repos/asf/kyuubi.git


The following commit(s) were added to refs/heads/debug by this push:
     new 3ad42d523 addr comments
3ad42d523 is described below

commit 3ad42d523658e1f97b2f1fa65519642580103e81
Author: Kent Yao <y...@apache.org>
AuthorDate: Mon Oct 23 17:26:32 2023 +0800

    addr comments
---
 .../src/main/scala/org/apache/kyuubi/util/reflect/ReflectUtils.scala  | 2 +-
 .../test/scala/org/apache/kyuubi/util/reflect/ReflectUtilsSuite.scala | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git 
a/kyuubi-util-scala/src/main/scala/org/apache/kyuubi/util/reflect/ReflectUtils.scala
 
b/kyuubi-util-scala/src/main/scala/org/apache/kyuubi/util/reflect/ReflectUtils.scala
index 9019ff187..f985e4067 100644
--- 
a/kyuubi-util-scala/src/main/scala/org/apache/kyuubi/util/reflect/ReflectUtils.scala
+++ 
b/kyuubi-util-scala/src/main/scala/org/apache/kyuubi/util/reflect/ReflectUtils.scala
@@ -89,7 +89,7 @@ object ReflectUtils {
     } catch {
       case e: Exception =>
         throw new RuntimeException(
-          s"$clz does not have $methodName method or incorrect arguments",
+          s"$clz does not have 
$methodName${argClasses.map(_.getName).mkString("(", ", ", ")")}",
           e)
     }
   }
diff --git 
a/kyuubi-util-scala/src/test/scala/org/apache/kyuubi/util/reflect/ReflectUtilsSuite.scala
 
b/kyuubi-util-scala/src/test/scala/org/apache/kyuubi/util/reflect/ReflectUtilsSuite.scala
index 1eff78a11..8f0090d5d 100644
--- 
a/kyuubi-util-scala/src/test/scala/org/apache/kyuubi/util/reflect/ReflectUtilsSuite.scala
+++ 
b/kyuubi-util-scala/src/test/scala/org/apache/kyuubi/util/reflect/ReflectUtilsSuite.scala
@@ -75,8 +75,8 @@ class ReflectUtilsSuite extends AnyFunSuite {
         "methodNotExists",
         (classOf[String], "arg1"),
         (classOf[String], "arg2"))
-    }("class org.apache.kyuubi.util.reflect.ObjectA$ does not have 
methodNotExists method or" +
-      " incorrect arguments")
+    }("class org.apache.kyuubi.util.reflect.ObjectA$ does not have 
methodNotExists(" +
+      "java.lang.String, java.lang.String)")
   }
 }