dlmarion commented on code in PR #5574:
URL: https://github.com/apache/accumulo/pull/5574#discussion_r2104570239
##########
core/src/main/java/org/apache/accumulo/core/util/Halt.java:
##########
@@ -30,25 +30,19 @@ public class Halt {
public static void halt(final String msg) {
// ACCUMULO-3651 Changed level to error and added FATAL to message for
slf4j compatibility
- halt(0, new Runnable() {
- @Override
- public void run() {
- log.error("FATAL {}", msg);
- }
- });
+ halt(0, "FATAL " + msg, null);
}
public static void halt(final String msg, int status) {
- halt(status, new Runnable() {
- @Override
- public void run() {
- log.error("FATAL {}", msg);
- }
- });
+ halt(status, "FATAL " + msg, null);
}
- public static void halt(final int status, Runnable runnable) {
+ public static void halt(final int status, String msg, Runnable runnable) {
Review Comment:
I added the Optional and added overloaded methods to Halt to reduce the
amount of noise in the calling code.
--
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
