+1
If you are confident, that's good for me
On Tue, Mar 6, 2018, 7:24 AM Sanne Grinovero <sanne(a)hibernate.org> wrote:
On 6 March 2018 at 11:21, Christian Beikov
<christian.beikov(a)gmail.com>
wrote:
> Hey,
>
> I'm fine with reverting the patch as well, though I just fixed the issue
> Andrea found and am confident about the solution now.
Hi Christian, that's great! Let's go with your fix then.
Thanks,
Sanne
>
>
> Mit freundlichen Grüßen,
> ------------------------------------------------------------------------
> *Christian Beikov*
> Am 06.03.2018 um 11:57 schrieb Guillaume Smet:
>> Hi,
>>
>> So, AFAICS, Andrea found another failing case for the new PR.
>>
>> I would vote for reverting the original patch [
>>
https://hibernate.atlassian.net/browse/HHH-11544] for now and let
Christian
>> the time to think of a proper fix. I'm a bit worried we will miss
something
>> if we try to find a fix in a hurry.
>>
>> AFAICS, the original issue was a bit of a corner case so better get the
>> other more common cases working.
>>
>> Anyone against reverting the original patch? Christian?
>>
>
> _______________________________________________
> hibernate-dev mailing list
> hibernate-dev(a)lists.jboss.org
>
https://lists.jboss.org/mailman/listinfo/hibernate-dev
_______________________________________________
hibernate-dev mailing list
hibernate-dev(a)lists.jboss.org
https://lists.jboss.org/mailman/listinfo/hibernate-dev