vinothchandar commented on a change in pull request #1159: [HUDI-479] Eliminate 
or Minimize use of Guava if possible
URL: https://github.com/apache/incubator-hudi/pull/1159#discussion_r398325200
 
 

 ##########
 File path: 
hudi-common/src/main/java/org/apache/hudi/common/util/ReflectionUtils.java
 ##########
 @@ -90,16 +99,58 @@ public static Object loadClass(String clazz, Object... 
constructorArgs) {
   }
 
   /**
-   * Return stream of top level class names in the same class path as 
passed-in class.
-   * 
-   * @param clazz
+   * Scans all classes accessible from the context class loader
+   * which belong to the given package and subpackages.
+   *
+   * @param clazz class
+   * @return Stream of Class names in package
    */
-  public static Stream<String> getTopLevelClassesInClasspath(Class clazz) {
+  public static Stream<String> getTopLevelClassesInClasspath(Class<?> clazz) {
 
 Review comment:
   this method is just used by the dummy bundle main classes right.. Ideally, 
like to simplify this down the line. if you agree, we can raise a JIRA? 

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to