Christian Beikov
Christian Beikov
Please rebase the PR and make sure you annotate the classes which you move with `@Internal`.
>But as you can see now, I reverted the move and changed its visibility to public. But why do you need this in the first place? The class is only...
If you provide a PR for it, then yeah.
Hey @jai-yadav and thanks for your contribution. A few comments: * Sometimes you are trying to access the SQL through the registry even when the SQL is given as argument,...
I understand the desire to fix this, but I just wanted to note that the only usages of this method are passing in `Types.CHAR` so the faulty branch will never...
How about wrapping an `InputStream` to make sure closing only happens at the end of a TX when we do resource cleanup in Hibernate? If we see that the `InputStream`...
>my concern is the impact of the new `IdentityHashMap` for small/medium results where the original solution had not real perf issues We could try to allocate the `IdentityHashMap` only when...
>IMO conceptually we'd need a https://en.wikipedia.org/wiki/Bloom_filter >Not keen to add a dependency though - could we check how close we get approximating it via an identity hashmap ? Yeah, either...
I was just looking into this, but it seems newer Maven and/or Java versions (11 update 9) are not affected anymore. At least I couldn't reproduce the error. Please continue...