Dans l'ensemble, c'est bon. Sauf que en général, on utilise l'objet Set plutôt que List.

Ensuite, car Hibernate pompe tout, il est intéressant de couper avec Fetch.
Fetch.Eager indique de tout...