ctubbsii commented on code in PR #5409:
URL: https://github.com/apache/accumulo/pull/5409#discussion_r1999725093
##########
server/manager/src/main/java/org/apache/accumulo/manager/upgrade/Upgrader12to13.java:
##########
@@ -141,6 +141,8 @@ private void createFateTable(ServerContext context) {
} catch (Exception e) {
Review Comment:
This is a very broad catch block. It seems like we are looking for a very
specific stack trace here. This catch block could be made more narrow to just
check for the type(s) we are actually looking for that would have this as a
cause in this situation.
--
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]
