Java 在@Transactional method可以';在(获取LazyInitializationException)之后不能使用
我看了很多关于Java 在@Transactional method可以';在(获取LazyInitializationException)之后不能使用,java,spring,hibernate,transactions,lazy-loading,Java,Spring,Hibernate,Transactions,Lazy Loading,我看了很多关于lazyiinitializationexception和@Transactional的其他问题,但没有看到任何对我们的案例有帮助的东西 我为没有系统地调查此事而道歉。这不是我的代码,但它暂时落入我的手中 我们的代码如下所示: @Entity class TaskSchedule { // fields that make up a task schedule Date nextRunTime; } 据我所知,在为任务中的第一项调用@Transactional方
lazyiinitializationexception
和@Transactional
的其他问题,但没有看到任何对我们的案例有帮助的东西
我为没有系统地调查此事而道歉。这不是我的代码,但它暂时落入我的手中
我们的代码如下所示:
@Entity
class TaskSchedule {
// fields that make up a task schedule
Date nextRunTime;
}
据我所知,在为
任务
中的第一项调用@Transactional
方法doTask
后,大量对象,包括任务
中剩余项的所有其他惰性计划
列表,通过调用取消设置会话
从会话中删除。这意味着任务
上的后续迭代会崩溃,因为它们的惰性列表不再具有关联的会话
实际上,我还没有编写这段代码,但最近它开始失败(在重新加载数据库之后)。如果从会话中删除对象,我不明白为什么它首先会起作用。@Transactional annotation只适用于公共方法。将@Transactional添加到runScheduledTasks()方法并从getScheduledTasks()中删除一个,或者将getScheduledTasks()公开 请阅读以下说明:
好的,现在您有了自我调用。请阅读我之前发布的链接中的所有答案(请注意,这也取决于您是否使用AOP代理,但我假设您正在使用)。在当前代码段中,您需要使用@Transactional注释runScheduledTasks(),并从getScheduledTasks()中删除@Transactional,因为它是冗余的,并且您不需要事务来读取数据库。同样,正如Nyar Thar提到的,您不需要用@Transactional注释实体方法 根据我的经验,@Transactional注释通常用于服务层。如果您希望在需要创建始终打开新事务的服务方法时更新新事务中的每个任务
public class TaskService {
@Transactional(propagation = REQUIRES_NEW)
public void updateTask(Task task) {
task.doTask();
}
}
然后在监视器工作
//in this case you don't need @Transactional here
public void runScheduledTasks() {
....
List<Task> tasks = getScheduledTasks();
for (Task task : tasks) {
taskService.updateTask(task);
}
}
//在这种情况下,此处不需要@Transactional
public void runScheduledTasks(){
....
列表任务=getScheduledTasks();
for(任务:任务){
taskService.updateTask(任务);
}
}
为什么不runScheduledTasks()
带@Transactional
注释?我以前没有在实体级方法中看到过@Transactional
。更改方法可见性是我在为SO编辑代码时引入的错误。原始版本使用了public
方法。谢谢你发现它!还有其他建议吗?该死!同样,getScheduledTasks
只在同一个类中,因为我编辑了代码以使StackOverflow更清晰。既然我知道私有调用和自调用如何影响@Transactional
,我就知道以后不要这样做了。
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
}
}
}
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)
public class TaskService {
@Transactional(propagation = REQUIRES_NEW)
public void updateTask(Task task) {
task.doTask();
}
}
//in this case you don't need @Transactional here
public void runScheduledTasks() {
....
List<Task> tasks = getScheduledTasks();
for (Task task : tasks) {
taskService.updateTask(task);
}
}