Hibernate Lazy Collection löschen

[OnDemand]

Hallo zusammen,

habe folgende Collections, welche beim löschen des Products nicht gelöscht werden. Muss ich die Produkt-Objekte erst alle rausholen und dann das löschen der Bilder einzeln anstoßen? Möchte ungern Eager Loading machen weil das die Performance sehr beeinflusst.

public class Product {
    
    @OneToMany(orphanRemoval = true, cascade = {CascadeType.ALL})
    @LazyCollection(LazyCollectionOption.TRUE)
    @JoinColumn(name = "product_id")
    private List<ProductImage> productImages = new ArrayList<>();
}

public class ProductService{
    ...
        
     List<Product> list = productRepository.findByxxx(xxx);
        for (Product product : list) {
            productRepository.delete(product);
     }
}

Die Produkte werden alle gelöscht, aber die Images bleiben in ihrer Tabelle.

 

[Dimax]

das löschen der Bilder einzeln anstoßen?

Es ist manchmal so das das Löschen von Sätzen in der DB nur mit primary key möglich ist. Also prüfen ob die Bilder nach primary key gelöscht werden.

 

[OnDemand]

In der Images Tabelle ist der primaryKey die ID (nicht product_id - das ist der ForeignKey). Das löschen funktioniert ja grundsätzlich, wenn es NICHT LAZY geladen wird. Das muss also daran liegen, dass diese Collection nicht geladen wird, wenn man das Product löscht.

 

[Dimax]

Ist die Image Tabelle an die Tabelle Produkt referenziert? Wenn ja und das eine MySql Tabelle ist ,dann kann man ON DELETE CASCADE einstellen.

 

[Oneixee5]

Zunächst sollte @LazyCollection(LazyCollectionOption.TRUE) nicht verwendet werden, @LazyCollection ist Hibernate spezifisch und es sollte nur auf JPA-Standards zurückgegriffen werden.
@OneToMany(orphanRemoval = true, cascade = CascadeType.ALL...ist im Prinzip ausreichend, da lazy fetching default ist:

OneToMany (javax.persistence-api 2.2 API)

docs.jboss.org

Wenn man aber ganz sicher sein will: @OneToMany(orphanRemoval = true, cascade = CascadeType.ALL, fetch = FetchType.LAZY, mappedBy = "product")

 

[OnDemand]

Ahaa! Danke, das ist eine gute Erklärung. Aber wozu brauch ich das mappedBy?

 

[OnDemand]

Wenn ich das mit dem Mapped by mache, ist meine product_id NULL in der DB. Was mach ich n hier falsch?

class Product{
    @OneToMany(orphanRemoval = true, cascade = {CascadeType.ALL},  fetch = FetchType.LAZY, mappedBy = "product")
    private List<ProductImage> productImages = new ArrayList<>();
}

class ProductImage{
    @ManyToOne(fetch = FetchType.LAZY)
    @JoinColumn(name = "product_id")
    private Product product;
}

Ich setzte über den Setter eine List an ProductImages, sehe die im Debugger auch, aber er speicher die ID nicht rein. Warum ist das so?

Edit: Wenn ich dem Image auch noch das Produkt setze, wirds gehen. Das hatte ich schon mal fällt mir grad ein, dann komme ich früher oder später wieder zu dem Teil wo ich "detached object..." Fehler bekomme. Hab ich noch nie kapiert.

 

[Oneixee5]

mappedBy bezieht sich auf das Feld, dem die Beziehung gehört. Erforderlich, sofern die Beziehung nicht unidirektional ist.

Hibernate One to Many Annotation Tutorial | Baeldung

In this tutorial we'll have a look at the one-to-many mapping using JPA annotations with a practical example.

www.baeldung.com

@JoinColumn Annotation Explained | Baeldung

Learn what JPA's @JoinColumn annotation is all about

www.baeldung.com

Für eine umfassende Beurteilung möglicher Fehler ist zu wenig Code vorhanden. Man kann sich das erzeugte SQL ausgeben lassen und analysieren. So kommt man meist schon auf die Ursache des Problems. Das die product_id auf null gesetzt wird könnte mehrere Ursachen haben.