AnnotatedElementUtils leniently ignores TypeNotPresentExceptions (just like AnnotationUtils)

Also refines logIntrospectionFailure to just log at debug level for meta-annotation introspection failures.

Issue: SPR-12889

spring-core/src/main/java/org/springframework/core/annotation/AnnotatedElementUtils.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2002-2014 the original author or authors.
+ * Copyright 2002-2015 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -188,6 +188,7 @@ public class AnnotatedElementUtils {
 			Processor<T> processor, Set<AnnotatedElement> visited, int metaDepth) {
 
 		if (visited.add(element)) {
+			try {
 				Annotation[] annotations =
 						(traverseClassHierarchy ? element.getDeclaredAnnotations() : element.getAnnotations());
 				for (Annotation annotation : annotations) {
@@ -224,6 +225,10 @@ public class AnnotatedElementUtils {
 					}
 				}
 			}
+			catch (Exception ex) {
+				AnnotationUtils.logIntrospectionFailure(element, ex);
+			}
+		}
 		return null;
 	}
 
@@ -232,7 +237,7 @@ public class AnnotatedElementUtils {
 	 * Callback interface used to process an annotation.
 	 * @param <T> the result type
 	 */
-	private static interface Processor<T> {
+	private interface Processor<T> {
 
 		/**
 		 * Called to process the annotation.

spring-core/src/main/java/org/springframework/core/annotation/AnnotationUtils.java

@@ -613,7 +613,7 @@ public abstract class AnnotationUtils {
 				value = ((Class<?>) value).getName();
 			}
 			else if (value instanceof Class[]) {
-				Class<?>[] clazzArray = (Class[]) value;
+				Class<?>[] clazzArray = (Class<?>[]) value;
 				String[] newValue = new String[clazzArray.length];
 				for (int i = 0; i < clazzArray.length; i++) {
 					newValue[i] = clazzArray[i].getName();
@@ -726,14 +726,30 @@ public abstract class AnnotationUtils {
 	}
 
 
-	private static void logIntrospectionFailure(AnnotatedElement annotatedElement, Exception ex) {
+	/**
+	 * Log an introspection failure (in particular {@code TypeNotPresentExceptions}) -
+	 * before moving on, pretending there were no annotations on this specific element.
+	 * @param element the element that we tried to introspect annotations on
+	 * @param ex the exception that we encountered
+	 */
+	static void logIntrospectionFailure(AnnotatedElement element, Exception ex) {
 		Log loggerToUse = logger;
 		if (loggerToUse == null) {
 			loggerToUse = LogFactory.getLog(AnnotationUtils.class);
 			logger = loggerToUse;
 		}
+		if (element instanceof Class && Annotation.class.isAssignableFrom((Class<?>) element)) {
+			// Meta-annotation lookup on an annotation type
+			if (logger.isDebugEnabled()) {
+				logger.debug("Failed to introspect meta-annotations on [" + element + "]: " + ex);
+			}
+		}
+		else {
+			// Direct annotation lookup on regular Class, Method, Field
 			if (loggerToUse.isInfoEnabled()) {
-			loggerToUse.info("Failed to introspect annotations on [" + annotatedElement + "]: " + ex);
+				logger.info("Failed to introspect annotations on [" + element + "]: " + ex);
+			}
+
 		}
 	}
 
@@ -793,9 +809,10 @@ public abstract class AnnotationUtils {
 		}
 
 		@SuppressWarnings("unchecked")
-		private void process(AnnotatedElement annotatedElement) {
-			if (this.visited.add(annotatedElement)) {
-				for (Annotation ann : annotatedElement.getAnnotations()) {
+		private void process(AnnotatedElement element) {
+			if (this.visited.add(element)) {
+				try {
+					for (Annotation ann : element.getAnnotations()) {
 						if (ObjectUtils.nullSafeEquals(this.annotationType, ann.annotationType())) {
 							this.result.add((A) ann);
 						}
@@ -807,6 +824,10 @@ public abstract class AnnotationUtils {
 						}
 					}
 				}
+				catch (Exception ex) {
+					logIntrospectionFailure(element, ex);
+				}
+			}
 		}
 
 		@SuppressWarnings("unchecked")