休眠：同时有两个或多个事务：事务已处于活动状态

Question

我有一个 REST API，当我几乎同时进行 POST 和 GET 时，我得到了这个异常：

 SEVERE: The RuntimeException could not be mapped to a response, re-throwing to the HTTP container
java.lang.IllegalStateException: Transaction already active
    at org.hibernate.engine.transaction.internal.TransactionImpl.begin(TransactionImpl.java:52)
    at org.hibernate.internal.AbstractSharedSessionContract.beginTransaction(AbstractSharedSessionContract.java:409)
    at sun.reflect.GeneratedMethodAccessor89.invoke(Unknown Source)
    at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
    at java.lang.reflect.Method.invoke(Method.java:498)
    at org.hibernate.context.internal.ThreadLocalSessionContext$TransactionProtectionWrapper.invoke(ThreadLocalSessionContext.java:355)
    at com.sun.proxy.$Proxy58.beginTransaction(Unknown Source)
    at utils.HibernateSession.createTransaction(HibernateSession.java:15)
    at api.ConversationsREST.getMessages(ConversationsREST.java:128)

它们位于不同的类中，因此没有隐含的全局属性。

失败的线路是这一行：

HibernateSession hs = new HibernateSession();
hs.createTransaction(); // Crash

Wich 指的是我的类 HibernateSession：

public class HibernateSession {

    public Session session;

    public void createTransaction() {

        session = HibernateUtil.getSessionFactory().getCurrentSession(); //THIS WAS WRONG
 //EDIT:session = HibernateUtil.getSessionFactory().openSession(); //THIS IS RIGHT
        session.beginTransaction();
    }

    public void commitclose() {

        session.getTransaction().commit();
        session.close();

    }

    public void rollbackclose() {

        try {
            session.getTransaction().rollback();
            session.close();
        } catch (Exception hibernateexception) {
            hibernateexception.printStackTrace();
        }

    }

}

异常居然在线session.beginTransaction()

我总是做一个 hs.commitclose() 并且在 catch() 块和 404 中我总是做一个 rollbackclose();

问题是当我发出这样的 POST 消息时：

HibernateSession hs = new HibernateSession();
hs.createTransaction();
hs.session.save(whatever);
hs.commitclose();

返回 200 并没关系，但随后 GET 可能会崩溃，但上述异常除外。 当我创建一个新的 HibernateSession 实例时，为什么 Hibernate 似乎试图共享该事务？

这只发生在我在很短的时间内进行两个查询时（我猜想在另一个人的开始和提交之间开始一个事务）。所以我猜 Hibernate 认为会话属性是静态的或类似的东西......

提前感谢您的帮助！

编辑：应要求，HibernateUtil.java（问题一定不在这里，但可能有助于理解）：

package utils;

import org.hibernate.SessionFactory;
import org.hibernate.cfg.Configuration;

public class HibernateUtil {

    private static SessionFactory sessionFactory;

    public static SessionFactory getSessionFactory() {

        if (sessionFactory == null) {

            sessionFactory = build();

        }
        return sessionFactory;

    }

    private static SessionFactory build() {

        try {

            return new Configuration().configure().buildSessionFactory();

        } catch (Throwable ex) {

            System.err.println("Initial SessionFactory creation failed: " + ex);
            throw new ExceptionInInitializerError(ex);
        }

    }

}

Answer 1

尝试将代码重写为

public class HibernateSession {

    public Session session;
    int id;
    Transaction t = null;

    public void createTransaction() {
        session = HibernateUtil.getSessionFactory().getCurrentSession();
        t = session.beginTransaction();
    }

    public void commitclose() {
        t.commit();
        session.close();
    }

    public void rollbackclose() {

        try {
            t.rollback();
            session.close();
        } catch (Exception hibernateexception) {
            hibernateexception.printStackTrace();
        }

    }

}

t 的每个引用都需要进行非正常空检查。

Answer 2

根据您的 createTransaction 逻辑，您正在从 SessionFactory 获取当前会话并从中开始事务。

这就是问题所在。

假设您已经创建了HibernateSession 对象并开始了事务，但它仍在进行中。所以你还没有关闭交易。

现在您创建了另一个HibernateSession 并尝试启动事务，这样做会引发异常。

所以你的这个代码

public void createTransaction() {
    session = HibernateUtil.getSessionFactory().getCurrentSession();
    session.beginTransaction();
}

应该是这样的

public void createTransaction() {
    session = HibernateUtil.getSessionFactory().openSession();
    session.beginTransaction();
}

getSessionFactory().openSession() 总是会打开一个新会话，一旦您完成操作就必须关闭该会话。

getSessionFactory().getCurrentSession() 返回绑定到上下文的会话 - 您无需关闭它。

Answer 3

其实你的实际代码有很多缺点：

• 每次调用代码时，您都会创建 HibernateSession 类的新实例，因此您将拥有许多实例，试图访问同一个会话。
• HibernateSession 的每个实例都会在您调用 hs.createTransaction(); 时尝试创建一个新事务，这就是您在此行收到异常的原因，因为已经有打开的事务并且您正在尝试打开一个新的，因为你每次都打电话给session.beginTransaction();，而没有打电话给transaction.close();。

您可以在这里做的是使您的HibernateSessionclass 单例，因此只有一个实例可用，您可以查看Singleton Pattern implementation 以获取有关如何实现它的更多详细信息。

在createTransaction 方法中，最好将其命名为getTransaction 而不是仅仅调用session.beginTransaction(); 您需要获取当前事务（如果存在）您可以使用session.getTransaction() 方法检查它并确保包装try ...catch 块中的代码，您可以查看此 Hibernate session tutorial 了解更多详细信息。

Answer 4

你有一些设计相关的缺陷：

1. 你不应该使用getCurrentSession()

如 jboss documentation 中所述，您应该使用以下成语：

 Session sess = factory.openSession();
 Transaction tx;
 try {
     tx = sess.beginTransaction();
     //do some work
     ...
     tx.commit();
 }
 catch (Exception e) {
     if (tx!=null) tx.rollback();
     throw e;
 }
 finally {
     sess.close();
 }

1. 您应该遵循 CRUD 模式

为每个操作（创建、读取、更新、删除）在您的类中创建一个方法。每个方法都应创建自己的自己的会话和事务以使用，因为如this answer 所述：

session 不是线程安全的对象 - 不能被多个共享 线程。您应该始终使用“每个请求一个会话”或“一个 每个事务的会话”

你的方法有些有效，但也看看this tutorial中创建的DAO对象

解决方案：

您的代码失败的原因仅仅是因为您之前与数据库的交互仍在进行中。这就是为什么您执行finally{ session.close() } - 以确保在离开该方法时会话已关闭。我假设在某些时候您的commit() 没有成功，并且您的休眠状态一直处于此后未关闭的事务/会话中。

要解决这个问题，您应该在代码中插入一次 session.close()，执行它，然后实现我建议的 try-catch-finally 块，以确保它在将来被关闭。

Answer 5

我刚刚换了

session = HibernateUtil.getSessionFactory().getCurrentSession();

与

session = HibernateUtil.getSessionFactory().openSession();

我正在访问已打开的工厂会话。