Jakarta migration: Agroal, Hibernate ORM, Hibernate Search...

[gsmet]

For now, I build a fork of Agroal locally while waiting for the release.

[quarkus-bot]

/cc @ebullient, @jmartisk, @manovotn, @maxandersen, @radcortez, @Sanne

[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 end up with dot

This message is automatically generated by a bot.

[gsmet]

@glefloch any chance you could have a look at the Gradle failure? I have no idea how what I did could have triggered the issue and the setup of the test is a bit complex.

[gsmet]

BTW, I can reproduce it locally so there's a real issue. But no idea why the dependency is now popping up.

From what I can see this particular build brings implementation("io.quarkus:quarkus-agroal-deployment") from ext-a so its not surprising that Agroal is around in the deployment dir but maybe I'm missing something.

[glefloch]

@gsmet yes I will have a look this afternoon.

[gsmet]

@glefloch cool, thanks!

[Sanne]

is that using your snapshot? Remember Gradle won't look into your local snapshots unless specifically allowed

…

On Tue, 12 Apr 2022, 12:25 Guillaume Smet, ***@***.***> wrote: @glefloch <https://github.com/glefloch> cool, thanks! — Reply to this email directly, view it on GitHub <#24879 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAAKMTKKBS2ID3XCCQMQVMDVEVMTPANCNFSM5TD6RKWA> . You are receiving this because you were mentioned.Message ID: ***@***.***>

[glefloch]

Looking at git blame it seems that narayana-deployment and agraoal-deployment dependencies where added by mistake when testing the validation task of io.quarkus.extension Gradle plugin.

In that test we check that agroal is not transitively imported when depending on panache-reactive (we add an issue on hat), however, I don't understand why it only starts failing now ... I pushed a fix that remove those dependencies.

[gsmet]

thanks @glefloch