Re: [hibernate-dev] HHH-13317

2019-09-11 Thread Gail Badner
I see. HHH-13317 is a valid mapping, and the types do match, so it is a separate issue. On Tue, Sep 10, 2019 at 8:54 PM Luis Barreiro wrote: > Hello Gail, > > The code in question deals with the detection of bi-directional > associations and only affects that feature. > > What happens currently

Re: [hibernate-dev] HHH-13317

2019-09-10 Thread Luis Barreiro
Hello Gail, The code in question deals with the detection of bi-directional associations and only affects that feature. What happens currently is that in certain situations fields are matched (based on the value of mappedBy) but the types don't actually match, leading to either the failure of

Re: [hibernate-dev] HHH-13317

2019-09-09 Thread Gail Badner
Hi Luis, IIUC, HHH-13446 deals with a mappedBy="" value that is invalid. I'm confused why the test in the PR is passing. Shouldn't an exception be thrown? HHH-13317 deals with a mappedBy value that is an association in an embeddable, which is valid. I know that Hibernate does not support a l

Re: [hibernate-dev] HHH-13317

2019-09-05 Thread Gail Badner
I'll take a look. Thanks Luis! On Thu, Sep 5, 2019 at 3:02 PM Luis Barreiro wrote: > Hello Gail, > > Without looking with much detail, this seems to be the same as HHH-13446. > > There is a PR for that for some time now: > https://github.com/hibernate/hibernate-orm/pull/2919 > > If it's the sa

Re: [hibernate-dev] HHH-13317

2019-09-05 Thread Luis Barreiro
Hello Gail, Without looking with much detail, this seems to be the same as HHH-13446. There is a PR for that for some time now: https://github.com/hibernate/hibernate-orm/pull/2919 If it's the same issue, it should be fixed. While Javassist fails when enhancing the classes, ByteBuddy succeeds