You are viewing a plain text version of this content. The canonical link for it is here.
Posted to server-dev@james.apache.org by bt...@apache.org on 2018/03/23 09:26:58 UTC
[4/9] james-project git commit: MAILBOX-325 Logging failed
transactions
MAILBOX-325 Logging failed transactions
Project: http://git-wip-us.apache.org/repos/asf/james-project/repo
Commit: http://git-wip-us.apache.org/repos/asf/james-project/commit/336d1f94
Tree: http://git-wip-us.apache.org/repos/asf/james-project/tree/336d1f94
Diff: http://git-wip-us.apache.org/repos/asf/james-project/diff/336d1f94
Branch: refs/heads/master
Commit: 336d1f94bac006cfe871fcec2c9094e2141bee44
Parents: 157d070
Author: benwa <bt...@linagora.com>
Authored: Thu Mar 22 09:47:52 2018 +0700
Committer: benwa <bt...@linagora.com>
Committed: Fri Mar 23 16:25:48 2018 +0700
----------------------------------------------------------------------
backends-common/jpa/pom.xml | 4 ++++
.../java/org/apache/james/backends/jpa/TransactionRunner.java | 6 ++++++
2 files changed, 10 insertions(+)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/james-project/blob/336d1f94/backends-common/jpa/pom.xml
----------------------------------------------------------------------
diff --git a/backends-common/jpa/pom.xml b/backends-common/jpa/pom.xml
index dbab212..6587c5d 100644
--- a/backends-common/jpa/pom.xml
+++ b/backends-common/jpa/pom.xml
@@ -29,5 +29,9 @@
<artifactId>openjpa</artifactId>
<version>2.4.2</version>
</dependency>
+ <dependency>
+ <groupId>org.slf4j</groupId>
+ <artifactId>slf4j-api</artifactId>
+ </dependency>
</dependencies>
</project>
http://git-wip-us.apache.org/repos/asf/james-project/blob/336d1f94/backends-common/jpa/src/main/java/org/apache/james/backends/jpa/TransactionRunner.java
----------------------------------------------------------------------
diff --git a/backends-common/jpa/src/main/java/org/apache/james/backends/jpa/TransactionRunner.java b/backends-common/jpa/src/main/java/org/apache/james/backends/jpa/TransactionRunner.java
index 71299c4..0b3c914 100644
--- a/backends-common/jpa/src/main/java/org/apache/james/backends/jpa/TransactionRunner.java
+++ b/backends-common/jpa/src/main/java/org/apache/james/backends/jpa/TransactionRunner.java
@@ -26,8 +26,13 @@ import javax.persistence.EntityManagerFactory;
import javax.persistence.EntityTransaction;
import javax.persistence.PersistenceException;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+
public class TransactionRunner {
+ private static final Logger LOGGER = LoggerFactory.getLogger(TransactionRunner.class);
+
private final EntityManagerFactory entityManagerFactory;
public TransactionRunner(EntityManagerFactory entityManagerFactory) {
@@ -42,6 +47,7 @@ public class TransactionRunner {
runnable.accept(entityManager);
transaction.commit();
} catch (PersistenceException e) {
+ LOGGER.warn("Could not execute transaction", e);
if (transaction.isActive()) {
transaction.rollback();
}
---------------------------------------------------------------------
To unsubscribe, e-mail: server-dev-unsubscribe@james.apache.org
For additional commands, e-mail: server-dev-help@james.apache.org