Rest: shorten the names of the generated classes

[mkouba]

Description

There seems to be no reason for the names to be so looong.

For example io.quarkus.resteasy.reactive.server.runtime.exceptionmappers.AuthenticationFailedExceptionMapper$GeneratedExceptionHandlerFor$AuthenticationFailedException$OfMethod$handle. The simple name has 111 chars in this particular case.

The ArC > Beans view in the Dev UI does not look good. Also every call site of this class must store this name in the constant pool ;-).

Zulip discussion: https://quarkusio.zulipchat.com/#narrow/stream/187038-dev/topic/Loooong.20names.20of.20generated.20classes.20in.20resteasy-reactive

Implementation ideas

No response

[quarkus-bot]

You added a link to a Zulip discussion, please make sure the description of the issue is comprehensive and doesn't require accessing Zulip

_{This message is automatically generated by a bot.}

[cescoffier]

Any good solution outside computing a hash of the name?

[mkouba]

Any good solution outside computing a hash of the name?

A computed hash is fine IMO.

[cescoffier]

Hashes work but make debugging harder and logger names useless. But if that's the only solution, we must live with it.

[mkouba]

Hashes work but make debugging harder and logger names useless. But if that's the only solution, we must live with it.

Well, I don't think we use loggers in these generated classes. As for the debugging - the base class is usually good enough to find out where to start. In case of the example above - io.quarkus.resteasy.reactive.server.runtime.exceptionmappers.AuthenticationFailedExceptionMapper is useful, but $GeneratedExceptionHandlerFor$AuthenticationFailedException$OfMethod$handle not that much.

[quarkus-bot]

/cc @FroMage (rest), @stuartwdouglas (rest)

[maxandersen]

besides devui would these not show up in stacktraces too? all for not too long names but shouldn't also just be a hashid as name brings meaning :)

[cescoffier]

If we can keep the meaningful part and append a hash, it would provide a good experience.

[geoand]

#44418 would make the name something like: io.quarkus.resteasy.reactive.server.runtime.exceptionmappers.AuthenticationFailedExceptionMapper$ExceptionMapper$123456