JPA / Hibernate Spring @Transactional vs. JOIN FETCH

Question

i'am facing with interesting solution of LazyInitializationException. To prevent this (on OneToMany oder ManyToMany) one known solution is, to use JOIN FETCH Query. You can see one of severals examples her: https://thoughts-on-java.org/best-practices-for-many-to-many-associations-with-hibernate-and-jpa/

Other easier solution is, to use @Transactional from Spring. For example like this:

@DeleteMapping(value ="/product/{tagId}")
    @ResponseBody
    @Transactional
    public String deleteProductWithoutRelation(@PathVariable String product, Model model) {     
        Optional<Product> pr = productService.selectProduct(product);
        if (pr.isPresent()) {
            tag.get().getCustomer().size(); //usualy throws LazyInitializationException, 
                                            //without JOIN-FETCH Statment or @Transactional
        return deletedTagId;    
    }

Of course, you can place @Transactional of some method from repository service, to encapsulate this solution. So which Advantages or Disadvantages of both solutions are here?

controller is not good place for transactions, see https://stackoverflow.com/questions/23118789/why-we-shouldnt-make-a-spring-mvc-controller-transactional — User9123, Mar 03 '20 at 19:15
@Transactional doesn't close the session until the call ends degrading performance. — Adithya, Mar 03 '20 at 19:26

score 6 · Accepted Answer · answered Mar 04 '20 at 11:21

There's a couple of things we need to untangle here.

@Transactional means nothing more than Spring makes sure to open up a database connection(+ transaction) and closes it again. That's it.
When you select an entity containing a lazy field, you are essentially saying: I am selecting "some" fields from my entity, except the lazy one.
If you, however, need that lazy field later on because you are trying to access it in your views (.html, .ftl, .jsp, whatever) you need to issue another select to the database to fetch it.
The problem: At that point, if you are outside of a @Transactional method you do not have a database connection open anymore, hence the LazyInitException.
To sum up: Your fetch makes sure to issue 1 select for all the data. If you do NOT do that, you need an open database connection/transaction, which @Transactional gives you.

Recommendation: You should try and fetch all the data you need to render a view, with appropriate JPQL/Criteria/SQL statements and not rely on re-selecting lazy-fields too much.

Thx. for reply, i've inspect hibernate queries carefully and saw, that @Transactional executes one query more for lazy-field. With appropriate join-fetch query you have only one query. — Roma Kap, Mar 07 '20 at 13:16

JPA / Hibernate Spring @Transactional vs. JOIN FETCH

1 Answers1