Hi Sir,

In Hibernate Annotations 3.3.0 GA Reference Guide (page 20), there's the following:

"Troop has a bidirectional one to many relationship with Soldier through the troop property. You don't have to
(must not) define any physical mapping in the mappedBy side.

To map a bidirectional one to many, with the one-to-many side as the owning side, you have to remove the
mappedBy element and set the many to one @JoinColumn as insertable and updatable to false. This solution is
obviously not optimized and will produce some additional UPDATE statements."

Can anyone comment if the mentioned solution is not optimized, what will be the optimize solution?

Please advise.

the optimal solution is to use the previous mapping (ie with mappedBy)

But why the documentation put it as "You don't have to
(must not) define any physical mapping in the mappedBy side. "

Sounds like we should not use "mappedby" but your advice is using mappedby is the optimum solution.

I'm confuse, do you mind to clarify further?

physycal mapping == @JoinTable or @JoinColumn
mappedBy is a logical mapping