ctubbsii commented on code in PR #3402:
URL: https://github.com/apache/accumulo/pull/3402#discussion_r1196811749
##########
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:
Not sure. Seems like some source could have been trying to catch stuff not
possible to be thrown, and compiler wouldn't have cared, but now with the
narrower exception, compiler will probably flag those as an attempt to catch
something not thrown. Not sure about binary, but not expecting binary
compatibility across major versions anyway.
--
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]
