Skip to content

WELD-2812 Improve Invoker exception messages#3137

Merged
manovotn merged 1 commit intoweld:6.0from
manovotn:weld2812_60
Mar 19, 2025
Merged

WELD-2812 Improve Invoker exception messages#3137
manovotn merged 1 commit intoweld:6.0from
manovotn:weld2812_60

Conversation

@manovotn
Copy link
Member

6.0 version of #3133

If an invoker invocation causes a ClassCastException,
NullPointerException or IllegalArgumentexception, validate the instance
and arguments to see if the exception was caused by passing invalid
values. If so, discard the original exception and throw a new exception
with more information.

Doing these checks only when the invocation throws an exception avoids
the overhead of doing these checks before every successful invocation.
@manovotn manovotn changed the title Improve Invoker exception messages WELD-2812 Improve Invoker exception messages Mar 19, 2025
@manovotn manovotn merged commit b90fcc3 into weld:6.0 Mar 19, 2025
12 checks passed
@manovotn manovotn deleted the weld2812_60 branch March 19, 2025 18:11
@Azquelt
Copy link
Contributor

Azquelt commented Apr 2, 2025

Can we have a 6.0.x release with this change in it sometime soon please?

@manovotn
Copy link
Member Author

manovotn commented Apr 3, 2025

Can we have a 6.0.x release with this change in it sometime soon please?

Sure, 6.0.2.Final is now syncing with Central

@Azquelt
Copy link
Contributor

Azquelt commented Apr 3, 2025

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants