Christian Beikov
Christian Beikov
Thanks for the fix @korlov42, this also solved query planning issues of correlated subqueries that we recently faced. Looking forward to the next release!
IMO we should leave `fetch` and `optional` to `@Basic`. Also, the `length` should maybe be optional with a reasonable default that allows vendor flexibility i.e. value `0`.
The default could be the maximum possible size a DB allows. Since Oracle requires to specify array sizes, but other databases support just "unlimited" (within the bounds of row format...
Like I wrote, every DB other than Oracle supports unlimited size by default or doesn't even allow specifying a size limit in the column definition. To restrict the size, we'd...
Yeah, `Class targetClass() default void.class;` is probably appropriate.
This PR also addresses some issues with date conversion between java.time other APIs where the same epoch millisecond might mean different dates for various reasons. It's also worth to mention...
Yeah, ignore them. I'm working on fixing CI as part of https://github.com/hibernate/hibernate-orm/pull/11345
Thanks for notifying me about this. I updated the GitHub action step versions and rebased your PR.
Looks good to me, can you please squash the commits and use `[#2010] unknown ids on window subquery` as commit message?
>Squashed and pushed, but isn't that usually done during merge? Thanks. It can be done during merge, but then I think that authorship attribution might be messed up. Either way,...