Martin Kouba

Results 269 comments of Martin Kouba

> I'm investigating the `AnnotationServletTestCase` failure. It's related to this [piece of code](https://github.com/quarkusio/quarkus/blob/main/independent-projects/arc/runtime/src/main/java/io/quarkus/arc/impl/ArcContainerImpl.java#L227-L237). ~~[This build step](https://github.com/quarkusio/quarkus/pull/28112/files#diff-598212c9f99cfb60963ca530475525e701b973a2c603d1d7bd22d02f3ed581d6R28-R52) should solve the issue. I've also created https://github.com/quarkusio/quarkus/issues/28115 to fix/revisit the non-standard behavior of...

I'm not an expert in tracing and rest client but the use case seems weird to me. A `@Scheduled` method is definitely not a good match (and definitely not an...

For the sake of completeness, it is possible to remove this information via `quarkus.arc.detect-unused-false-positives=false`. > Perhaps we need a custom encoding, or just represent them as strings and do some...

FTR after we merged the micro optimizations from the https://github.com/quarkusio/quarkus/pull/18567 the resulting set of removed beans will usually take a few tens of kilobytes of heap (8 KB for the...

Well, we can still do better and save few tens of kilobytes of heap ;-). I'd like to keep this issue open but the priority is very low.

So I managed to load the removed beans lazily but then realized that many extensions unfortunately use the _"try-to-get a bean instance programmatically"_ pattern during initialization (and handling the potential...

Hm, it seems that we don't log a warning if at least one bean matches. So maybe it would be more meaningful to only emit a warning for `CDI.current().select(Foo.class).get()` and...

> @mkouba I see you updated the example you mentioned above. But I assume your comment about us needing to rewrite the rest of the instances still applies? Well, any...

> Regarding the loading getting triggered when our extensions aren't careful about it - would there be a way to verify noone is doing that? No way I'm aware of....

> Perhaps that's too much work, but I'm wondering. Yes, I think that it's too much. Keep in mind that we have `quarkus.arc.detect-unused-false-positives=false` and then this lazy loading. That's IMO...