You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by "keith-turner (via GitHub)" <gi...@apache.org> on 2023/05/16 20:57:39 UTC

[GitHub] [accumulo] keith-turner commented on a diff in pull request #3402: Improve some exception handling

keith-turner commented on code in PR #3402:
URL: https://github.com/apache/accumulo/pull/3402#discussion_r1195688330


##########
core/src/main/java/org/apache/accumulo/core/client/PluginEnvironment.java:
##########
@@ -163,7 +163,7 @@ interface Configuration extends Iterable<Entry<String,String>> {
    * @param className Fully qualified name of the class.
    * @param base The expected super type of the class.
    */
-  <T> T instantiate(String className, Class<T> base) throws Exception;
+  <T> T instantiate(String className, Class<T> base) throws ReflectiveOperationException;

Review Comment:
   Will this change cause any compatibility issues?  Thinking it will not, but not completely sure. Seems like its source compatible, not sure about binary.



-- 
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-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org