Catching Throwable is necessary since such code might throw a LinkageError.

As for logging the exception, that is effectively unnecessary since any attempt to access the value for that attribute later will result in an exception. For example, see AnnotationAttributes#assertNotException(String, Object).

I am therefore closing this PR.

Having said that, now that I've looked at AnnotationReadingVisitorUtils and AnnotationAttributes#assertNotException(String, Object) again, I noticed that the latter should actually check if (attributeValue instanceof Throwable) instead of if (attributeValue instanceof Exception).

@jhoeller, what do you think about making that change in AnnotationAttributes?

Reopening in order to make a decision regarding this.

Superseded by gh-23424