From d0fcd4a13abbc17abf84a06ef6f9479a9f50d064 Mon Sep 17 00:00:00 2001 From: Rob Bygrave Date: Fri, 4 Feb 2022 15:55:00 +1300 Subject: [PATCH] #2549 - Fix DefaultPersister to expose non RuntimeExceptions on insert/update/delete. Not hide NoClassDefFoundError etc --- .../server/persist/DefaultPersister.java | 11 ++++------- 1 file changed, 4 insertions(+), 7 deletions(-) diff --git a/ebean-core/src/main/java/io/ebeaninternal/server/persist/DefaultPersister.java b/ebean-core/src/main/java/io/ebeaninternal/server/persist/DefaultPersister.java index 07410134b..4e4fc1039 100644 --- a/ebean-core/src/main/java/io/ebeaninternal/server/persist/DefaultPersister.java +++ b/ebean-core/src/main/java/io/ebeaninternal/server/persist/DefaultPersister.java @@ -81,8 +81,7 @@ public final class DefaultPersister implements Persister { int rc = request.executeOrQueue(); request.commitTransIfRequired(); return rc; - - } catch (RuntimeException e) { + } catch (Throwable e) { request.rollbackTransIfRequired(); throw e; } finally { @@ -383,8 +382,7 @@ public final class DefaultPersister implements Persister { req.resetDepth(); req.commitTransIfRequired(); req.flushBatchOnCascade(); - - } catch (RuntimeException ex) { + } catch (Throwable ex) { req.rollbackTransIfRequired(); throw ex; } finally { @@ -420,8 +418,7 @@ public final class DefaultPersister implements Persister { req.resetDepth(); req.commitTransIfRequired(); req.flushBatchOnCascade(); - - } catch (RuntimeException ex) { + } catch (Throwable ex) { req.rollbackTransIfRequired(); throw ex; } finally { @@ -559,7 +556,7 @@ public final class DefaultPersister implements Persister { req.flushBatchOnCascade(); return rows; - } catch (RuntimeException ex) { + } catch (Throwable ex) { req.rollbackTransIfRequired(); throw ex; } finally {