Fix Redis dev mode

[stuartwdouglas]

Fixes #18669

[gsmet]

@stuartwdouglas I'm not sure I understand why it didn't work before? I also wouldn't be surprised we use the exact same pattern elsewhere.

[stuartwdouglas]

There was an @PreDestroy on something that was not a CDI bean

[gsmet]

Ah yeah OK, I missed that. I think it used to be a CDI bean but got changed at some point. I wonder if we could detect than somehow and throw at least a warning if not an error. @mkouba WDYT?

[mkouba]

I wonder if we could detect than somehow and throw at least a warning if not an error.

Detect what exactly? That there's a @PreDestroy declared on a class which is not a bean? We could definitely do that. But I'm not quite sure we should throw an error because it comes from javax.annotation and thus can be used by other frameworks and libraries, i.e. it's not CDI-only.

[stuartwdouglas]

Yea, it's not really practical. It's possible to have classes in extensions that are only sometimes registered as beans.