Should be ManagedExecutor in the Reactive Messaging Doc

[murphye]

There is an error on the doc here:
https://quarkus.io/guides/reactive-messaging#implementing-fire-and-forget-interactions

Not correct:
@Inject Executor executor;

Correct:
@Inject ManagedExecutor executor;

[murphye]

Opening issue for website.

[gsmet]

Reopening as the doc are maintained here.

@cescoffier @FroMage could you check and fix if needed?

[FroMage]

Mmm, I wonder if it's required. ManagedExecutor is an Executor, so it would work unless there are other Executor beans, and I'm not sure there are in Quarkus. I'd have to actually try this code out to know.

[FroMage]

@cescoffier I assume you did try that example code?

[gsmet]

Well, if it doesn't hurt, I would be in favor of using ManagedExecutor.

Having some other Executor bean could be quite frequent.

[loicmathieu]

If it's still needed, I can take care of this one.

[cescoffier]

I just opened a PR.