Discussion :

[rozwel]

Je débute avec JPA mais j'utilise Hibernate depuis longtemps. Là j'ai un souci d'optimisation des selects.
J'ai les 2 classes suivantes:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
 
@Entity
public class Bill {
    @Id
    @GeneratedValue
    private long id;
    private long totalBytes;
    private long uploadedBytes;
    private String fileName;
    private Date billingDate;
    private Date importDate;
    @ManyToOne
    @Fetch(FetchMode.JOIN)    
    private Customer customer;
    @ManyToOne
    private Format format;
    private String filePathOnServer;
}
 
@Entity
public class Customer {
    @Id
    @GeneratedValue
    private long id;
    private String name;
    private String number;
    private Address postAddress;
    @OneToMany
    private Set<Bill> bills = new HashSet<Bill>();
}

Ensuite j'ai un DAO JPA qui me sert notamment à récupérer tous les bills:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
 
  public List<Bill> findAll() {
        return entityManager.createQuery("select b from Bill b").getResultList();
    }

Et avant même que j'appelle getCustomer() sur bill, Hibernate a déjà exécuté N+1 requêtes: 1 pour les bills, et N pour le customer de chaque bill. Or, je me serais attendu à ce qu'il n'exécute qu'une seule requête avec un join.

Est-ce qu'il n'est pas censé faire ça par défaut? Sinon comment je peux forcer Hibernate à le faire?

Je précise que je suis sur une base MySQL 5.

[al3alwa]

salut,

tu dois faire comme cela dans la classe Customer pour avoir une relation bidirectionelle entre les deux entities

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
 
 @OneToMany(mappedBy="customer")
    private Set<Bill> bills = new HashSet<Bill>();

comme cela tu auras qu'un clé etranger dans la table Bill et pas les deux (cle de jointure + table de jointure bill_costumer)

essaie cela et normalement JPA te fera qu'une seule requete avec un left outer join...
A+

[al3alwa]

juste un truc dans le cas d'une relation ManyToOne bidirectionnelle la collection la plus performante et Collection avec un @IdCollection et pas dans ton cas le Set...

A+

[OButterlin]

Et avant même que j'appelle getCustomer() sur bill, Hibernate a déjà exécuté N+1 requêtes: 1 pour les bills, et N pour le customer de chaque bill. Or, je me serais attendu à ce qu'il n'exécute qu'une seule requête avec un join.

Est-ce qu'il n'est pas censé faire ça par défaut? Sinon comment je peux forcer Hibernate à le faire?

Regarde du côté de FetchType.LAZY

[rozwel]

Je ne pense pas: FetchType.LAZY décalera la requête sur customer à quand je fais l'appel à bill.getCustomer().

Par contre j'ai essayé de modifier les annotations pour améliorer les choses:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
 
@Entity
public class Bill {
    @Id
    @GeneratedValue
    private long id;
    private long totalBytes;
    private long uploadedBytes;
    private String fileName;
    private Date billingDate;
    private Date importDate;
    @ManyToOne
    private Customer customer;
    @ManyToOne
    private Format format;
    private String filePathOnServer;
}
 
@Entity
public class Customer {
    @Id
    @GeneratedValue
    private long id;
    private String name;
    private String number;
    private Address postAddress;
    @OneToMany(mappedBy = "customer")
    private Collection<Bill> bills = new HashSet<Bill>();
}

Effectivement, la table de mapping a disparu, par contre mon log me dit toujours que la requête sur Customer est séparée.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
 
Hibernate: select bill0_.id as id2_, bill0_.billingDate as billingD2_2_, bill0_.customer_id as customer8_2_, bill0_.fileName as fileName2_, bill0_.filePathOnServer as filePath4_2_, bill0_.format_id as format9_2_, bill0_.importDate as importDate2_, bill0_.totalBytes as totalBytes2_, bill0_.uploadedBytes as uploaded7_2_ from Bill bill0_ where 1=1 order by bill0_.id
Hibernate: select customer0_.id as id0_0_, customer0_.name as name0_0_, customer0_.number as number0_0_, customer0_.addressLine1 as addressL4_0_0_, customer0_.addressLine2 as addressL5_0_0_, customer0_.city as city0_0_, customer0_.postCode as postCode0_0_ from Customer customer0_ where customer0_.id=?

[OButterlin]

Si tu veux qu'il fasse la jonction directement dans la requête initiale sur Bill, il faudrait utiliser "fetch join" dans la requête, dans ce genre

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
entityManager.createQuery("select b from Bill b fetch join customer ...")

[rozwel]

Merci beaucoup, celle là a marché:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
select b from Bill b join fetch b.customer

Je ne comprends toujours pas pourquoi @Fetch(FetchMode.JOIN) ne fait pas ça automatiquement, mais au moins j'ai quelque chose qui fonctionne maintenant.

Merci.