Skip to content

Commit

Permalink
check that the new SparkOutOfMemoryError is not a JVM failure
Browse files Browse the repository at this point in the history
  • Loading branch information
sumwale committed Mar 27, 2020
1 parent fbacaa8 commit e5ecb5d
Showing 1 changed file with 2 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -317,7 +317,8 @@ public static void setSkipOOMEForThread(boolean skip) {
public static boolean isJVMFailureError(Error err) {
// all VirtualMachineErrors are not fatal to the JVM, in particular
// StackOverflowError is not
if (err instanceof OutOfMemoryError) {
if (err instanceof OutOfMemoryError &&
!err.getClass().getName().contains("SparkOutOfMemoryError")) {
if (SKIP_OOME.get() == Boolean.TRUE) return false;
// ignore OOMEs thrown by Spark
String message = err.getMessage();
Expand Down

0 comments on commit e5ecb5d

Please sign in to comment.