deniskuzZ commented on code in PR #6146:
URL: https://github.com/apache/hive/pull/6146#discussion_r2451814486


##########
ql/src/java/org/apache/hadoop/hive/ql/exec/Utilities.java:
##########
@@ -4132,8 +4132,22 @@ public static boolean isPerfOrAboveLogging(HiveConf 
conf) {
   public static String jarFinderGetJar(Class klass) {
     Preconditions.checkNotNull(klass, "klass");
     ClassLoader loader = klass.getClassLoader();
+    return jarFinderGetJar(loader, klass.getName());
+  }
+
+  /**
+   * Returns the full path to the Jar containing the class.
+   *
+   * @param loader
+   *          the classloader instance to scan for jars.
+   * @param className
+   *          the name of the class to look for.
+   *
+   * @return path to the Jar containing the class.
+   */
+  public static String jarFinderGetJar(ClassLoader loader, String className) {

Review Comment:
   arg order should be reversed



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