HHH-13103 - Allow Hibernate Types to get access to the current configuration properties

[vladmihalcea]

https://hibernate.atlassian.net/browse/HHH-13103

[vladmihalcea]

@sebersole The reason why I think it's worth to implement this feature is that we are currently configuring the Java and SQL Type Descriptors in the constructor of the Type instances.

With this feature, we could allow the user to provide a specific constructor which takes the TypeBootstrapContext where we can pass the current SessionFactory properties or some other info we might need in future:

public ArrayType(TypeBootstrapContext typeBootstrapContext) {
    super( VarcharTypeDescriptor.INSTANCE, ArrayTypeDescriptor.INSTANCE );
    this.settings = typeBootstrapContext.getConfigurationSettings();
}

This way, we can get access to the current properties and the user can customize the custom Type object creation using the Hibernate properties.

One use case where I wanted this future is to customize the Jackson ObjectMapper used by the JSON Types. Currently, I can only do that via loading the hibernate.properties file externally, but not via the SessionFactory properties that might be passed by Spring or Spring Boot. This is the original issue that made me realize we could benefit from this new feature.

Let me know what you think.

[cberg-zalando]

@vladmihalcea I guess you will have to resolve the merge conflict before this moves any forward.

[vladmihalcea]

@dreab8 I rebased this PR. From the upvotes this PR got, I think it's very useful and worth integrating.

Do you think it could be integrated in the next release? It's a small change and shouldn't be risky at all.

[sebersole]

Not a fan of creating a new TypeBootstrapContext every time. I'd pass it the TypeConfiguration and be done with it, or have TypeConfiguration implement TypeBootstrapContext (after making it an interface - which it ought to be anyway imo)

[sebersole]

Or better yet, have TypeFactory be the TypeBootstrapContext

[sebersole]

I can work on this change

[vladmihalcea]

Thanks.

[sebersole]

I've left a few comments..

[sebersole]

Applied upstream with discussed change