Add properties for statement-batch-size and fetch.batch-size in hibernate-reactive

[holomekc]

Fixes #16640

-HibernateReactiveProcessor adjusted so that it supports new property for quarkus.hibernate-orm.fetch.batch-size
-HibernateReactiveProcessor adjusted so that it supports property for quarkus.hibernate-orm.jdbc.statement-batch-size

[quarkus-bot]

Thanks for your pull request!

The title of your pull request does not follow our editorial rules. Could you have a look?

• title should not contain an issue number (use 'Fix Allow * expression for quarkus:add-extension #1234' in the description instead)

This message is automatically generated by a bot.

[holomekc]

/cc @DavideD, @Sanne, @gavinking

[Sanne]

@DavideD could you review this? Are these properties honoured in Hibernate Reactive?

Ideally someone should add a test as well. @holomekc would you be able?

[DavideD]

Yeah sorry, I've been a bit busy with other issues, It looks fine at first glance but I will have better look at it soon

[holomekc]

@Sanne
Sorry for my late answer. I did not really have time to create some tests. To be honest it feels like more configuration should be adjusted and I am not sure if I have all insights here what your plans are and if the two settings cover everything that is missing.

I just noticed this during development where I was confused about the sql statements sent by hibernate.

If I find the time to do so I can try to add some tests. But I do not want to promise this because we are quite busy at the moment.

[gsmet]

Hi!

This will need a rebase as things have been partially implemented here 49886c8#diff-0e6dba65e9fb565f0f6a85e937bcab1f9574091bfe5323a4965f085bbab07b5b already.

[Sanne]

@DavideD I'll assign this to you so it doesn't get lost. Please could you 1)have a look 2)resolve the conflicts 3)merge it !
Thanks

[DavideD]

I've rebased the PR and sent a new one here: #18241

But I've realized that, unrelated to Quarkus, Hibernate Reactive currently ignores the value set by the property.