Search code examples
javaspringspring-data-jpaspring-datahibernate-mapping

Transactional annotation doesn't solve org.hibernate.LazyInitializationException


I'm writing a simple Spring Data JPA application. I use MySQL database. There are two simple tables:

  1. Department
  2. Employee

Each employee works in some department (Employee.department_id).

I have two entity classes:

@Entity
public class Department {
    @GeneratedValue(strategy = GenerationType.IDENTITY)
    @Id
    private Long id;

    @Basic(fetch = FetchType.LAZY)
    @OneToMany(mappedBy = "department")
    List<Employee> employees;
}

@Entity
public class Person {
    @GeneratedValue(strategy = GenerationType.IDENTITY)
    @Id
    private Long id;

    @ManyToOne
    @JoinColumn
    private Department department;
}

The repositories:

@Repository
public interface DepartmentRepository extends JpaRepository<Department, Long> {}

@Repository
public interface EmployeeRepository extends JpaRepository<Employee, Long> {}

There is a DepartmentService class:

@Service
@Transactional
public class DepartmentService {
    @Autowired
    DepartmentRepository departmentRepository;

    @Transactional
    List<Department> getAll() {
        departmentRepository.findAll();
    }

    @Transactional
    void getEmployees() {
        for(Department department: getAll()) {
            List<Employee> employees = department.getEmployees();
            for(Employee employee: employees)
                System.out.println(employee);
        }
    }
}

And the main class (I don't need web, so I use CommandLineRunner to run the application as a console application):

@SpringBootApplication
public class EmployeeDB implements CommandLineRunner {
    @Autowired
    DepartmentService departmentService;

    public static void main(String[] args) {
        SpringApplication.run(EmployeeDB.class, args);
    }

    @Override
    public void run(String... args) throws Exception {
        departmentService.getEmployees();
    }
}

I added @Transactional before the class DepartmentService and also before all its methods, but I still keep getting the following error:

 org.hibernate.LazyInitializationException: failed to lazily initialize a collection of role: employeeDB.Department.employees, could not initialize proxy - no Session

Stacktrace:

Caused by: org.hibernate.LazyInitializationException: failed to lazily initialize a collection of role: employeeDB.Department.employees, could not initialize proxy - no Session
    at org.hibernate.collection.internal.AbstractPersistentCollection.throwLazyInitializationException(AbstractPersistentCollection.java:602) ~[hibernate-core-5.4.2.Final.jar:5.4.2.Final]
    at org.hibernate.collection.internal.AbstractPersistentCollection.withTemporarySessionIfNeeded(AbstractPersistentCollection.java:217) ~[hibernate-core-5.4.2.Final.jar:5.4.2.Final]
    at org.hibernate.collection.internal.AbstractPersistentCollection.initialize(AbstractPersistentCollection.java:581) ~[hibernate-core-5.4.2.Final.jar:5.4.2.Final]
    at org.hibernate.collection.internal.AbstractPersistentCollection.read(AbstractPersistentCollection.java:148) ~[hibernate-core-5.4.2.Final.jar:5.4.2.Final]
    at org.hibernate.collection.internal.PersistentBag.iterator(PersistentBag.java:303) ~[hibernate-core-5.4.2.Final.jar:5.4.2.Final]
    at employeeDB.DepartmentService.getEmployees(DepartmentService.java:25) ~[main/:na]
    at employeeDB.EmployeeDB.run(EmployeeDB.java:41) [main/:na]
    at org.springframework.boot.SpringApplication.callRunner(SpringApplication.java:795) [spring-boot-2.3.3.RELEASE.jar:2.3.3.RELEASE]
    ... 5 common frames omitted

Why do I get this error? How can I solve it?


Solution

  • Summarizing all the answers and comments above:

    This happens because method DepartmentService::getEmployees is not public.

    I added public before getEmployees() and this sloved the problem.

    @Transactional annotation is powered by AOP, then only public methods could be handled by proxy.

    This means that @Transactional is not actually applied to this method and the session is already closed an the moment of reading lazy collection.