On Mon, Jul 25, 2016 at 2:10 PM, Emmanuel Bernard
<emman...@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
_______________________________________________
hibernate-dev mailing list
hibernate-dev@lists.jboss.org
https://lists.jboss.org/mailman/listinfo/hibernate-dev

Reply via email to