ctubbsii commented on code in PR #5876:
URL: https://github.com/apache/accumulo/pull/5876#discussion_r2342367340
##########
core/src/main/java/org/apache/accumulo/core/spi/scan/SimpleScanDispatcher.java:
##########
@@ -152,14 +152,10 @@ public ScanDispatch dispatch(DispatchParameters params) {
}
}
- switch (scanInfo.getScanType()) {
- case MULTI:
- return multiDispatch;
- case SINGLE:
- return singleDispatch;
- default:
- throw new IllegalArgumentException("Unexpected scan type " +
scanInfo.getScanType());
Review Comment:
I'm okay with this PR as-is, if you've checked that we're not explicitly
trying to catch and handle these removed IllegalStateException cases anywhere,
and if we are, that they are appropriately updated to handle
NullPointerException instead.
--
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-unsubscr...@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
