brianloss commented on a change in pull request #1891:
URL: https://github.com/apache/accumulo/pull/1891#discussion_r567310639
##########
File path:
core/src/main/java/org/apache/accumulo/core/client/PluginEnvironment.java
##########
@@ -158,6 +167,18 @@
*/
<T> T instantiate(String className, Class<T> base) throws Exception;
+ /**
+ * Loads a class using Accumulo's system classloader.
+ *
+ * @param className
+ * Fully qualified name of the class.
+ * @param base
+ * The expected super type of the class.
+ *
+ * @since 2.1.0
+ */
+ <T> Class<? extends T> loadClass(String className, Class<T> base) throws
ClassNotFoundException;
Review comment:
Fair enough. I really just needed the context-loading variant for
TableLoadBalancer. But that really just needs a method to get the context name
for the table configuration, so I added that method to BalancerEnvironment and
removed the loadClass methods from here.
----------------------------------------------------------------
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:
[email protected]
