wForget commented on code in PR #7081:
URL: https://github.com/apache/kyuubi/pull/7081#discussion_r2128097971
##########
kyuubi-util-scala/src/main/scala/org/apache/kyuubi/util/reflect/ReflectUtils.scala:
##########
@@ -64,6 +66,27 @@ object ReflectUtils {
}
}
+ /**
+ * Get a method with the given name and argument classes from the given
class.
+ * @param clz the class to get the method from
+ * @param methodName the method name from declared field names
+ * @param argClasses the classes of the arguments
+ * @return an unbound method that can be invoked later
+ */
+ def getMethod(clz: Class[_], methodName: String, argClasses: Class[_]*):
UnboundMethod = {
Review Comment:
> this does not simplify much code, can we inline it?
Makes sense, changed.
--
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: notifications-unsubscr...@kyuubi.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscr...@kyuubi.apache.org
For additional commands, e-mail: notifications-h...@kyuubi.apache.org
