diff --git a/extensions/spark/kyuubi-spark-authz/src/test/resources/log4j2-test.xml b/extensions/spark/kyuubi-spark-authz/src/test/resources/log4j2-test.xml index 7aaf820ad11..70940728128 100644 --- a/extensions/spark/kyuubi-spark-authz/src/test/resources/log4j2-test.xml +++ b/extensions/spark/kyuubi-spark-authz/src/test/resources/log4j2-test.xml @@ -35,7 +35,7 @@ - + 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 08916b8d150..f985e406705 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 @@ -60,11 +60,7 @@ object ReflectUtils { } } catch { case e: Exception => - val candidates = - (clz.getDeclaredFields ++ clz.getFields).map(_.getName).distinct.sorted - throw new RuntimeException( - s"Field $fieldName not in $clz [${candidates.mkString(",")}]", - e) + throw new RuntimeException(s"$clz does not have $fieldName field", e) } } @@ -92,14 +88,8 @@ object ReflectUtils { } } catch { case e: Exception => - val candidates = - (clz.getDeclaredMethods ++ clz.getMethods) - .map(m => s"${m.getName}(${m.getParameterTypes.map(_.getName).mkString(", ")})") - .distinct.sorted - val argClassesNames = argClasses.map(_.getName) throw new RuntimeException( - s"Method $methodName(${argClassesNames.mkString(", ")})" + - s" not found in $clz [${candidates.mkString(", ")}]", + 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 626aeebe226..8f0090d5dda 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,10 +75,8 @@ class ReflectUtilsSuite extends AnyFunSuite { "methodNotExists", (classOf[String], "arg1"), (classOf[String], "arg2")) - }("Method methodNotExists(java.lang.String, java.lang.String) not found " + - "in class org.apache.kyuubi.util.reflect.ObjectA$ " + - "[equals(java.lang.Object), field5(), field6(), getClass(), hashCode(), method5(), " + - "method6(), notify(), notifyAll(), toString(), wait(), wait(long), wait(long, int)]") + }("class org.apache.kyuubi.util.reflect.ObjectA$ does not have methodNotExists(" + + "java.lang.String, java.lang.String)") } }