Gavin King

icon indicating an email [email protected]

icon company IBM icon location Barcelona, Spain icon location Red Hat Distinguished Engineer at IBM. Creator of @Hibernate, @cdi-spec, and @ceylon. Working on @jakartaee Query, Persistence, and Data specifications.

Results 1136 comments of Gavin King

HHH-15422 Pick up `CurrentTenantIdentifierResolver` and `MultiTenantConnectionProvider` from BeanContainer if not explicit set

So as I was saying to Yoann on Zulip, I can see that `CurrentTenantIdentifierResolver` might legitimately be a contextual bean. I absolutely struggle to see how this could ever possibly...

HHH-15422 Pick up `CurrentTenantIdentifierResolver` and `MultiTenantConnectionProvider` from BeanContainer if not explicit set

> Most people will just expect a CDI integration to involve actual CDI beans and to comply with scopes I mean, just I don't see that. - Servlets support CDI...

HHH-15422 Pick up `CurrentTenantIdentifierResolver` and `MultiTenantConnectionProvider` from BeanContainer if not explicit set

> > as that class can implement `Configurable` and get injected with PU-specific config. > > OK, I get the point, I changed `useJpaCompliantCreation()` to return `true`, please review. Yeah,...

HHH-15422 Pick up `CurrentTenantIdentifierResolver` and `MultiTenantConnectionProvider` from BeanContainer if not explicit set

> I think there's a reasonable argument that `CurrentTenantIdentifierResolver` is a special case here Indeed, you could choose to view the `CurrentTenantIdentifierResolver` as precisely a kind of "bridge" between the...

HHH-18461 Fix native query doesn't respect `query.setFirstResult(0)`

Fine, then I'll let Christian make the call on this. To me it seems weird to change the SQL that was handwritten by the user _if we don't absolutely need...

possible mitigation for Oracle fetch size issue

Closing because this change was incorporated into #10633.

HHH-19113 introduce @NaturalIdClass

Note that it's going to take some work to finish this off, so it's not for 7.0.

HHH-19322 Test case for batched unidirectional one-to-many update

I have added additional tests here: #9971

Check deprecated settings properties

I guess ideally we would just accept whatever ORM accepts.

Check deprecated settings properties

I guess ideally we would just accept whatever ORM accepts.