On Mon, Jul 25, 2016 at 2:10 PM, Emmanuel Bernard
<emmanuel(a)hibernate.org> wrote:
I am having second thoughts on the collection case. When I look at
ORM's
test suite and AttributeOverrideTest + PropertyRecord, the property name
is not including the owning property (addresses in this case).
@AttributeOverrides({
@AttributeOverride(name = "size", column = @Column(name
= "SQUARE_FEET")),
@AttributeOverride(name = "tax", column = @Column(name
= "ASSESSMENT"))
})
@ElementCollection
public Set<PropertyInfo> unsortedParcels;
So we probably should not need to put "addresses" in the @Column for the
collection cases. It makes sense from an ORM PoV since we are talking about different
tables.
OK, so this means our current implementation is correct and we just
need to wait for 5.1.1 before merging pr/728.
--
Guillaume