eclipse-vertx / eclipse-vertx/vert.x

Consider removing NoStackTraceThrowable and NoStackTraceException

Open
#5,012 1 comment 4 reactions 0 assignees View on GitHub
enhancement
Dominant language
Java
Stars
14.7k
Forks
2.1k
Avg merge
2d 7h
Merged PRs (30d)
28

Description

There are several places in the codebase where `NoStackTraceThrowable` is used to convey a failure which does not need a stacktrace.

This isn't a big issue with the callback or futurized API, but when using the virtual threads threading model or Kotlin coroutines, it forces users to catch `Throwable` in their applications.

`VertxException` extends `RuntimeException` and can be created without filling the stacktrace. In fact, we already have a `NoStackTraceException` which extends `VertxException`.

In Vert.x 5, we should consider removing `NoStackTraceThrowable`.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.