Fix the non-default constructor mechanism of bytecode recording

[gastaldi]

• Fix the non-default constructor mechanism of bytecode recording
• Fixes Bug in Quarkus bytecode generation #31153
• Optimize bytecode recording test performance

[gastaldi]

Thanks to @Ladicek and @geoand who helped with the investigation and provided the proper fix!

[geoand]

I didn't do anything, you guys did all the work :)

[gastaldi]

@geoand well, you approved this PR, so that's something 😉

[Ladicek]

LGTM, obviously :-) It would be nice if @stuartwdouglas could also take a look!

[gastaldi]

@Ladicek looks like ArC didn't like that change:

Could not find parameters for constructor public io.smallrye.config.ConfigMappings$ConfigClassWithPrefix(java.lang.Class,java.lang.String) could not read field values [arg1, arg0]

[quarkus-bot]

Failing Jobs - Building 8a65238

Status	Name	Step	Failures	Logs	Raw logs
✔️	JVM Tests - JDK 11
✔️	JVM Tests - JDK 17
✖	JVM Tests - JDK 18	Build	⚠️ Check →	Logs	Raw logs
✖	MicroProfile TCKs Tests	Verify	Failures	Logs	Raw logs

Full information is available in the Build summary check run.

Failures

⚙️ MicroProfile TCKs Tests #

- Failing: tcks/microprofile-lra 

📦 tcks/microprofile-lra

✖ org.eclipse.microprofile.lra.tck.TckCancelOnTests.cancelOn301 line 120 - More details - Source on GitHub

java.lang.AssertionError: Response status on call to 'http://localhost:8081/lraresource-cancelon/cancelOn301' failed to match. expected:<301> but was:<503>
	at org.junit.Assert.fail(Assert.java:89)
	at org.junit.Assert.failNotEquals(Assert.java:835)

✖ org.eclipse.microprofile.lra.tck.TckCancelOnTests.cancelOnFamilyDefault4xx line 68 - More details - Source on GitHub

java.lang.AssertionError: Response status on call to 'http://localhost:8081/lraresource-cancelon/cancelOnFamilyDefault4xx' failed to match. expected:<400> but was:<503>
	at org.junit.Assert.fail(Assert.java:89)
	at org.junit.Assert.failNotEquals(Assert.java:835)

✖ org.eclipse.microprofile.lra.tck.TckCancelOnTests.cancelOnFamilyDefault5xx line 84 - More details - Source on GitHub

java.lang.AssertionError: Response status on call to 'http://localhost:8081/lraresource-cancelon/cancelOnFamilyDefault5xx' failed to match. expected:<500> but was:<503>
	at org.junit.Assert.fail(Assert.java:89)
	at org.junit.Assert.failNotEquals(Assert.java:835)

✖ org.eclipse.microprofile.lra.tck.TckCancelOnTests.notCancelOnFamily5xx line 140 - More details - Source on GitHub

java.lang.AssertionError: Response status on call to 'http://localhost:8081/lraresource-cancelon/notCancelOnFamily5xx' failed to match. expected:<500> but was:<503>
	at org.junit.Assert.fail(Assert.java:89)
	at org.junit.Assert.failNotEquals(Assert.java:835)

✖ org.eclipse.microprofile.lra.tck.TckCancelOnTests.cancelOnFamily3xx line 100 - More details - Source on GitHub

java.lang.AssertionError: Response status on call to 'http://localhost:8081/lraresource-cancelon/cancelOnFamily3xx' failed to match. expected:<303> but was:<503>
	at org.junit.Assert.fail(Assert.java:89)
	at org.junit.Assert.failNotEquals(Assert.java:835)

✖ org.eclipse.microprofile.lra.tck.TckCancelOnTests.cancelFromRemoteCall line 160 - More details - Source on GitHub

java.lang.AssertionError: Response status on call to 'http://localhost:8081/lraresource-cancelon/cancelFromRemoteCall' failed to match. expected:<200> but was:<503>
	at org.junit.Assert.fail(Assert.java:89)
	at org.junit.Assert.failNotEquals(Assert.java:835)

[gastaldi]

Test failures are unrelated and happen in current main branch

[gsmet]

If we want it backported, we will need specific PRs. It doesn't apply cleanly and I prefer having the author fix the conflicts given it's a sensitive matter.

[gastaldi]

I'll refrain from backporting this to 2.13 because it involves changes that would require a deeper analysis (like the introduction of RecordingAnnotationsUtil)

UPDATE: actually, I think that's quite trivial, let me check