Skip to content

Commit

Permalink
[HUDI-5350] Fix oom cause compaction event lost problem (#7408)
Browse files Browse the repository at this point in the history
Co-authored-by: hbg <bingeng.huang@shopee.com>
  • Loading branch information
2 people authored and Alexey Kudinkin committed Dec 14, 2022
1 parent 0e17e18 commit 1a2ffdd
Showing 1 changed file with 4 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -136,15 +136,15 @@ private <E extends Throwable> Runnable wrapAction(
}

private void handleException(Throwable t, ExceptionHook hook, Supplier<String> actionString) {
// if we have a JVM critical error, promote it immediately, there is a good
// chance the
// logging or job failing will not succeed any more
ExceptionUtils.rethrowIfFatalErrorOrOOM(t);
final String errMsg = String.format("Executor executes action [%s] error", actionString.get());
logger.error(errMsg, t);
if (hook != null) {
hook.apply(errMsg, t);
}
// if we have a JVM critical error, promote it immediately, there is a good
// chance the
// logging or job failing will not succeed any more
ExceptionUtils.rethrowIfFatalErrorOrOOM(t);
}

private Supplier<String> getActionString(String actionName, Object... actionParams) {
Expand Down

0 comments on commit 1a2ffdd

Please sign in to comment.