Skip performing content negotiation twice

[franz1981]

Description

quarkus/independent-projects/resteasy-reactive/server/runtime/src/main/java/org/jboss/resteasy/reactive/server/handlers/ClassRoutingHandler.java

Line 131 in af6d3d7

boolean matches = acceptHeaderMatches(target, accepts.get(i));

And

quarkus/independent-projects/resteasy-reactive/server/runtime/src/main/java/org/jboss/resteasy/reactive/server/handlers/FixedProducesHandler.java

Line 47 in af6d3d7

if (accept.contains(mediaTypeString) || accept.contains("*/*") || accept.contains(mediaTypeSubstring)) {

Both seems to verify that Accept and (some) mediaType matches, but the former just to send back a proper response (if unmatched), while the other to find the appropriate writer.

Implementation ideas

Probably would be better to merge them in one pass (on class routing) and just save somewhere the matched one, to allow the subsequent to pick it as a starting point to make faster verifying if the writer is fine.

Moreover, a first pass in the matching logic can be to find for wildcard types on accept, to skip any subsequent op.

[franz1981]

@geoand

[geoand]

👌

[geoand]

@franz1981 do you have some sample code that I can use to reproduce this and see its effect in a flamegraph?

[quarkus-bot]

/cc @FroMage (resteasy-reactive), @stuartwdouglas (resteasy-reactive)

[geoand]

I am trying a change now

[geoand]

#37646 should do the trick

[franz1981]

@geoand I think that's fine, buddy, thanks for looking into it: I'll answer in the PR than