DomGarguilo commented on code in PR #5876:
URL: https://github.com/apache/accumulo/pull/5876#discussion_r2345456533
##########
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 manually looked into each case here and it does not seem like any of the
calling code is explicitly expecting IllegalStateException.
--
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
