Re: [hibernate-dev] HHH-10162 Inheritance and L2 cache

2017-12-05 Thread Gail Badner
Adding Steve... On Thu, Mar 16, 2017 at 1:44 PM, Christian Beikov < christian.bei...@gmail.com> wrote: > Hey Gail, > > comments inline.. > > Am 15.03.2017 um 23:21 schrieb Gail Badner: > > Hi Christian, > > More comments below... > > On Fri, Mar 3, 2017 at 4:38 PM, Christian Beikov < > christian.

Re: [hibernate-dev] HHH-10162 Inheritance and L2 cache

2017-03-16 Thread Christian Beikov
Hey Gail, comments inline.. Am 15.03.2017 um 23:21 schrieb Gail Badner: > Hi Christian, > > More comments below... > > On Fri, Mar 3, 2017 at 4:38 PM, Christian Beikov > mailto:christian.bei...@gmail.com>> wrote: > > Thanks for the comments, I have updated the PR. > > I think that it is

Re: [hibernate-dev] HHH-10162 Inheritance and L2 cache

2017-03-15 Thread Gail Badner
Hi Christian, More comments below... On Fri, Mar 3, 2017 at 4:38 PM, Christian Beikov wrote: > Thanks for the comments, I have updated the PR. > > I think that it is important to have a complete fix and I am already > working on that, but this issue was specifically about the L2 cache, so I > w

Re: [hibernate-dev] HHH-10162 Inheritance and L2 cache

2017-03-03 Thread Christian Beikov
Thanks for the comments, I have updated the PR. I think that it is important to have a complete fix and I am already working on that, but this issue was specifically about the L2 cache, so I wanted to keep that stuff separate. Here are some other polymorphism related issues: * https://hibern

Re: [hibernate-dev] HHH-10162 Inheritance and L2 cache

2017-03-03 Thread Gail Badner
Hi Christian, I've added some comments to the PR. As I mentioned there, this appears to be a partial fix. If the application really requires the entity to implement the "correct" class, then this will still break for them when the entity is not in the cache. I'm not sure if a partial fix really

Re: [hibernate-dev] HHH-10162 Inheritance and L2 cache

2017-03-02 Thread Christian Beikov
I can, just wanted to hear some general opinions before creating the PR. Will do that tomorrow. Mit freundlichen Grüßen, *Christian Beikov* Am 02.03.2017 um 20:57 schrieb Gail Badner: > Hi Christian, > > You mentioned that

Re: [hibernate-dev] HHH-10162 Inheritance and L2 cache

2017-03-02 Thread Gail Badner
Hi Christian, You mentioned that you reproduced the issue, but I don't see a test case. Can you push your test case to a PR? Thanks, Gail On Thu, Mar 2, 2017 at 10:15 AM, Steve Ebersole wrote: > BTW, regarding the Hip Chat "Dev" room, I'd personally not rely on replies > from there. E.g. I ne

Re: [hibernate-dev] HHH-10162 Inheritance and L2 cache

2017-03-02 Thread Steve Ebersole
BTW, regarding the Hip Chat "Dev" room, I'd personally not rely on replies from there. E.g. I never saw your question there, and others likely did not either. FWIW we had all agreed on a block of time when we would try to be available in that room specifically to discuss such things. IIRC that w