Ladislav Thon

Results 352 comments of Ladislav Thon

OK, so in #989, I added `withThreadOffloadExecutor()` to specify an executor explicitly, in case the default one (provided by `StandaloneFaultTolerance.configure()`) is not the right one. I also realized that `FaultTolerance.call()`...

`FutureThreadPoolBulkhead` is not used in the programmatic API, only in the annotation-based API. I'm not going to add a timeout to bulkheads at the moment. It feels like a useful...

That's exactly the scope creep I want to avoid. Bulkhead (aka concurrency limiter) already has a concept of a queue, the concept of a timeout somewhat conflicts with that. It...

At this point, I'm not exactly sure we should do this. The synchronous exception handling at least guards against mistakes in our own code -- which are really easy to...

Full stack trace of the failing assert: https://github.com/quarkusio/quarkus/pull/9556#issuecomment-633299576

It may also fail in a different way: https://github.com/quarkusio/quarkus/pull/18387 Full stacktrace: ``` java.lang.AssertionError: Unexpected exception thrown from Future at org.testng.Assert.fail(Assert.java:85) at org.eclipse.microprofile.fault.tolerance.tck.util.Exceptions.expect(Exceptions.java:98) at org.eclipse.microprofile.fault.tolerance.tck.TimeoutUninterruptableTest.testTimeoutAsyncBulkhead(TimeoutUninterruptableTest.java:190) at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method) at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62) at...

This is a Jakarta EE 11 release, need to wait until we bump to latest CDI. Moving to draft for now.

Yeah, `String` is probably the first choice anyone would make. I'm not sure if it's the best choice, but it has all the nice qualities like: instantly familiar, immutable, usable...

Note to self: comes from this discussion: https://quarkusio.zulipchat.com/#narrow/stream/187030-users/topic/Microprofile.20.40Asynchronous.20when.20RequestScoped.20is.20not.20active