Browse Source

Raise beforeCompletion/afterCompletion exception log level to error

Closes gh-30776
pull/30915/head
Juergen Hoeller 3 years ago
parent
commit
f1567fb21a
  1. 4
      spring-tx/src/main/java/org/springframework/transaction/reactive/TransactionSynchronizationUtils.java
  2. 4
      spring-tx/src/main/java/org/springframework/transaction/support/TransactionSynchronizationUtils.java

4
spring-tx/src/main/java/org/springframework/transaction/reactive/TransactionSynchronizationUtils.java

@ -85,7 +85,7 @@ abstract class TransactionSynchronizationUtils {
public static Mono<Void> triggerBeforeCompletion(Collection<TransactionSynchronization> synchronizations) { public static Mono<Void> triggerBeforeCompletion(Collection<TransactionSynchronization> synchronizations) {
return Flux.fromIterable(synchronizations) return Flux.fromIterable(synchronizations)
.concatMap(TransactionSynchronization::beforeCompletion).onErrorContinue((t, o) -> .concatMap(TransactionSynchronization::beforeCompletion).onErrorContinue((t, o) ->
logger.debug("TransactionSynchronization.beforeCompletion threw exception", t)).then(); logger.error("TransactionSynchronization.beforeCompletion threw exception", t)).then();
} }
/** /**
@ -115,7 +115,7 @@ abstract class TransactionSynchronizationUtils {
Collection<TransactionSynchronization> synchronizations, int completionStatus) { Collection<TransactionSynchronization> synchronizations, int completionStatus) {
return Flux.fromIterable(synchronizations).concatMap(it -> it.afterCompletion(completionStatus)) return Flux.fromIterable(synchronizations).concatMap(it -> it.afterCompletion(completionStatus))
.onErrorContinue((t, o) -> logger.debug("TransactionSynchronization.afterCompletion threw exception", t)).then(); .onErrorContinue((t, o) -> logger.error("TransactionSynchronization.afterCompletion threw exception", t)).then();
} }

4
spring-tx/src/main/java/org/springframework/transaction/support/TransactionSynchronizationUtils.java

@ -108,7 +108,7 @@ public abstract class TransactionSynchronizationUtils {
synchronization.beforeCompletion(); synchronization.beforeCompletion();
} }
catch (Throwable ex) { catch (Throwable ex) {
logger.debug("TransactionSynchronization.beforeCompletion threw exception", ex); logger.error("TransactionSynchronization.beforeCompletion threw exception", ex);
} }
} }
} }
@ -172,7 +172,7 @@ public abstract class TransactionSynchronizationUtils {
synchronization.afterCompletion(completionStatus); synchronization.afterCompletion(completionStatus);
} }
catch (Throwable ex) { catch (Throwable ex) {
logger.debug("TransactionSynchronization.afterCompletion threw exception", ex); logger.error("TransactionSynchronization.afterCompletion threw exception", ex);
} }
} }
} }

Loading…
Cancel
Save