Question

我已经查看了很多关于LazyInitializationException和@Transactional的其他问题，并且没有看到任何似乎对我们案例有帮助的问题。

我为没有系统地调查它而道歉。这不是我的代码，但它暂时落入了我的手中......

我们的代码如下所示：

@Entity
class TaskSchedule {
    // fields that make up a task schedule
    Date nextRunTime;
}

@Entity
class Task {
    @OneToMany(fetch = FetchType.LAZY)
    List<TaskSchedule> schedules;

    @Transactional
    public void doTask() {
        // run the task

        for (TaskSchedule schedule : schedules) {  /* CRASH HAPPENS HERE */
            // set the schedule's next run time
        }
    }
}

@Component
public class TaskDao {
    @Transactional(readOnly = true)
    public List<Task> getScheduledTasks() {

        Calendar calendar = Calendar.getInstance();

        Criteria criteria = getSession().createCriteria(Task.class);
        criteria.add(Restrictions.le("runTime", calendar.getTime()));
        criteria.setResultTransformer(Criteria.DISTINCT_ROOT_ENTITY);

        List<Task> results = criteria.list();
        return results;
    }
}

public class MonitorJob {
    TaskDao taskDao;

    public void runScheduledTasks() {
        // start a read only transaction
        List<Task> tasks = taskDao.getScheduledTasks(); // query database
        // finish the transaction

        for (Task task : tasks) {
            // start a transaction
            task.doTask(); // populate lazy list
            // finish transaction
        }
    }
}

我们在迭代惰性列表LazyInitializationException时得到schedules异常：

org.hibernate.LazyInitializationException: failed to lazily initialize a collection of role: Task.schedules, no session or session was closed
        at org.hibernate.collection.AbstractPersistentCollection.throwLazyInitializationException(AbstractPersistentCollection.java:380)
        at org.hibernate.collection.AbstractPersistentCollection.throwLazyInitializationExceptionIfNotConnected(AbstractPersistentCollection.java:372)
        at org.hibernate.collection.AbstractPersistentCollection.initialize(AbstractPersistentCollection.java:365)
        at org.hibernate.collection.AbstractPersistentCollection.read(AbstractPersistentCollection.java:108)
        at org.hibernate.collection.PersistentBag.iterator(PersistentBag.java:272)
        at Task.doTask(Task.java:366)
        at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
        at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:39)
        at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25)
        at java.lang.reflect.Method.invoke(Method.java:597)
        at org.hibernate.proxy.pojo.javassist.JavassistLazyInitializer.invoke(JavassistLazyInitializer.java:197)
        at Task_$$_javassist_43.doTask(Task_$$_javassist_43.java)
        at MonitorJob.runScheduledTasks(MonitorJob.java:106)
        at sun.reflect.GeneratedMethodAccessor68.invoke(Unknown Source)
        at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25)
        at java.lang.reflect.Method.invoke(Method.java:597)
        at org.springframework.util.MethodInvoker.invoke(MethodInvoker.java:273)
        at org.springframework.scheduling.quartz.MethodInvokingJobDetailFactoryBean$MethodInvokingJob.executeInternal(MethodInvokingJobDetailFactoryBean.java:299)
        at org.springframework.scheduling.quartz.QuartzJobBean.execute(QuartzJobBean.java:111)
        at org.quartz.core.JobRunShell.run(JobRunShell.java:203)
        at org.quartz.simpl.SimpleThreadPool$WorkerThread.run(SimpleThreadPool.java:520)

据我所知，在为@Transactional中的第一项调用doTask方法tasks之后，包含了所有其他懒惰{{1}的所有对象schedules中剩余项目的列表会通过调用tasks从会话中删除。这意味着后续迭代超过unsetSession会崩溃，因为它们的惰性列表不再具有关联的会话。

我实际上并没有处理过这段代码，但它最近已经开始失败（在重新加载数据库之后）。如果对象从会话中删除，我无法理解为什么它首先会起作用。

Answer 1

好的，现在你得到了自我调用。请阅读我之前发布的链接中的所有答案（请注意，这也取决于您是否使用AOP代理 - 但我假设您是这样）。在当前的代码片段中，您需要使用@Transactional注释runScheduledTasks（）并从getScheduledTasks（）中删除@Transactional，因为它是多余的，并且您不需要事务来读取数据库。另外，正如A Nyar Thar所提到的，您不需要使用@Transactional注释您的实体方法。

另外根据我的经验，@ Transactal注释通常用于服务层。如果您希望在需要创建始终打开新事务的服务方法时更新新事务中的每个任务。

public class TaskService {

    @Transactional(propagation = REQUIRES_NEW)
    public void updateTask(Task task) {
        task.doTask();      
    }

}

然后在MonitorJob

//in this case you don't need @Transactional here
public void runScheduledTasks() {
        ....
        List<Task> tasks = getScheduledTasks();
        for (Task task : tasks) {
            taskService.updateTask(task);
        }
    }

Answer 2

@Transactional注释仅适用于公共方法。将@Transactional添加到runScheduledTasks（）方法并从getScheduledTasks（）中删除一个，或者将getScheduledTasks（）公开。

阅读本说明： Does Spring @Transactional attribute work on a private method?

在@Transactional方法之前创建的Hibernate对象之后无法使用（获取LazyInitializationException）

2 个答案: