Release v6.2.9

.github/workflows/ci.yml

@@ -21,7 +21,7 @@ jobs:
             toolchain: false
           - version: 21
             toolchain: true
-          - version: 25
+          - version: 23
             toolchain: true
         exclude:
           - os:

build.gradle

@@ -86,7 +86,7 @@ configure([rootProject] + javaProjects) { project ->
 	ext.javadocLinks = [
 			"https://docs.oracle.com/en/java/javase/17/docs/api/",
 			"https://jakarta.ee/specifications/platform/9/apidocs/",
-			"https://docs.hibernate.org/orm/5.6/javadocs/",
+			"https://docs.jboss.org/hibernate/orm/5.6/javadocs/",
 			"https://www.quartz-scheduler.org/api/2.3.0/",
 			"https://fasterxml.github.io/jackson-core/javadoc/2.14/",
 			"https://fasterxml.github.io/jackson-databind/javadoc/2.14/",
@@ -97,7 +97,7 @@ configure([rootProject] + javaProjects) { project ->
 			// TODO Uncomment link to JUnit 5 docs once we execute Gradle with Java 18+.
 			// See https://github.com/spring-projects/spring-framework/issues/27497
 			//
-			// "https://junit.org/junit5/docs/5.14.0/api/",
+			// "https://junit.org/junit5/docs/5.13.3/api/",
 			"https://www.reactive-streams.org/reactive-streams-1.0.3-javadoc/",
 			//"https://javadoc.io/static/io.rsocket/rsocket-core/1.1.1/",
 			"https://r2dbc.io/spec/1.0.0.RELEASE/api/",

framework-docs/modules/ROOT/pages/core/beans/classpath-scanning.adoc

@@ -1,26 +1,24 @@
 [[beans-classpath-scanning]]
 = Classpath Scanning and Managed Components
 
-Most examples in this chapter use XML to specify the configuration metadata that
-produces each `BeanDefinition` within the Spring container. The previous section
-(xref:core/beans/annotation-config.adoc[Annotation-based Container Configuration])
-demonstrates how to provide a lot of the configuration metadata through source-level
-annotations. Even in those examples, however, the "base" bean definitions are explicitly
-defined in the XML file, while the annotations drive only the dependency injection.
-
-This section describes an option for implicitly detecting the candidate components by
-scanning the classpath. Candidate components are classes that match against a filter
-criteria and have a corresponding bean definition registered with the container.
-This removes the need to use XML to perform bean registration. Instead, you can use
-annotations (for example, `@Component`), AspectJ type expressions, or your own
+Most examples in this chapter use XML to specify the configuration metadata that produces
+each `BeanDefinition` within the Spring container. The previous section
+(xref:core/beans/annotation-config.adoc[Annotation-based Container Configuration]) demonstrates how to provide a lot of the configuration
+metadata through source-level annotations. Even in those examples, however, the "base"
+bean definitions are explicitly defined in the XML file, while the annotations drive only
+the dependency injection. This section describes an option for implicitly detecting the
+candidate components by scanning the classpath. Candidate components are classes that
+match against a filter criteria and have a corresponding bean definition registered with
+the container. This removes the need to use XML to perform bean registration. Instead, you
+can use annotations (for example, `@Component`), AspectJ type expressions, or your own
 custom filter criteria to select which classes have bean definitions registered with
 the container.
 
 [NOTE]
 ====
 You can define beans using Java rather than using XML files. Take a look at the
-`@Configuration`, `@Bean`, `@Import`, and `@DependsOn` annotations for examples
-of how to use these features.
+`@Configuration`, `@Bean`, `@Import`, and `@DependsOn` annotations for examples of how to
+use these features.
 ====
 
 
@@ -832,10 +830,10 @@ definitions, there is no notion of bean definition inheritance, and inheritance
 hierarchies at the class level are irrelevant for metadata purposes.
 
 For details on web-specific scopes such as "`request`" or "`session`" in a Spring context,
-see xref:core/beans/factory-scopes.adoc#beans-factory-scopes-other[Request, Session, Application, and WebSocket Scopes].
-As with the pre-built annotations for those scopes, you may also compose your own scoping
-annotations by using Spring's meta-annotation approach: for example, a custom annotation
-meta-annotated with `@Scope("prototype")`, possibly also declaring a custom scoped-proxy mode.
+see xref:core/beans/factory-scopes.adoc#beans-factory-scopes-other[Request, Session, Application, and WebSocket Scopes]. As with the pre-built annotations for those scopes,
+you may also compose your own scoping annotations by using Spring's meta-annotation
+approach: for example, a custom annotation meta-annotated with `@Scope("prototype")`,
+possibly also declaring a custom scoped-proxy mode.
 
 NOTE: To provide a custom strategy for scope resolution rather than relying on the
 annotation-based approach, you can implement the
@@ -877,8 +875,7 @@ Kotlin::
 ----
 
 When using certain non-singleton scopes, it may be necessary to generate proxies for the
-scoped objects. The reasoning is described in
-xref:core/beans/factory-scopes.adoc#beans-factory-scopes-other-injection[Scoped Beans as Dependencies].
+scoped objects. The reasoning is described in xref:core/beans/factory-scopes.adoc#beans-factory-scopes-other-injection[Scoped Beans as Dependencies].
 For this purpose, a scoped-proxy attribute is available on the component-scan
 element. The three possible values are: `no`, `interfaces`, and `targetClass`. For example,
 the following configuration results in standard JDK dynamic proxies:

framework-docs/modules/ROOT/pages/core/beans/dependencies/factory-properties-detailed.adoc

@@ -85,11 +85,11 @@ element. The following example shows how to use it:
 
 [source,xml,indent=0,subs="verbatim,quotes"]
 ----
-	<bean id="collaborator" class="..." />
+	<bean id="theTargetBean" class="..."/>
 
-	<bean id="client" class="...">
+	<bean id="theClientBean" class="...">
 		<property name="targetName">
-			<idref bean="collaborator" />
+			<idref bean="theTargetBean"/>
 		</property>
 	</bean>
 ----
@@ -99,24 +99,28 @@ following snippet:
 
 [source,xml,indent=0,subs="verbatim,quotes"]
 ----
-	<bean id="collaborator" class="..." />
+	<bean id="theTargetBean" class="..." />
 
-	<bean id="client" class="...">
-		<property name="targetName" value="collaborator" />
+	<bean id="theClientBean" class="...">
+		<property name="targetName" ref="theTargetBean"/>
 	</bean>
 ----
 
 The first form is preferable to the second, because using the `idref` tag lets the
-container validate at deployment time that the referenced, named bean actually exists. In
-the second variation, no validation is performed on the value that is passed to the
-`targetName` property of the `client` bean. Typos are therefore only discovered (with most
+container validate at deployment time that the referenced, named bean actually
+exists. In the second variation, no validation is performed on the value that is passed
+to the `targetName` property of the `client` bean. Typos are only discovered (with most
 likely fatal results) when the `client` bean is actually instantiated. If the `client`
-bean is a xref:core/beans/factory-scopes.adoc[prototype] bean, this typo and the resulting
-exception may only be discovered long after the container is deployed.
+bean is a xref:core/beans/factory-scopes.adoc[prototype] bean, this typo and the resulting exception
+may only be discovered long after the container is deployed.
 
-NOTE: A common place (at least in versions earlier than Spring 2.0) where the `<idref/>`
-element brings value is in the configuration of xref:core/aop-api/pfb.adoc#aop-pfb-1[AOP interceptors]
-in a `ProxyFactoryBean` bean definition. Using `<idref/>` elements when you specify the
+NOTE: The `local` attribute on the `idref` element is no longer supported in the 4.0 beans
+XSD, since it does not provide value over a regular `bean` reference any more. Change
+your existing `idref local` references to `idref bean` when upgrading to the 4.0 schema.
+
+A common place (at least in versions earlier than Spring 2.0) where the `<idref/>` element
+brings value is in the configuration of xref:core/aop-api/pfb.adoc#aop-pfb-1[AOP interceptors] in a
+`ProxyFactoryBean` bean definition. Using `<idref/>` elements when you specify the
 interceptor names prevents you from misspelling an interceptor ID.
 
 

framework-docs/modules/ROOT/pages/testing/annotations/integration-spring/annotation-mockitobean.adoc

@@ -77,17 +77,6 @@ exactly one candidate bean exists.
 
 [TIP]
 ====
-As stated in the documentation for Mockito, there are times when using `Mockito.when()` is
-inappropriate for stubbing a spy – for example, if calling a real method on a spy results
-in undesired side effects.
-
-To avoid such undesired side effects, consider using
-`Mockito.doReturn(...).when(spy)...`, `Mockito.doThrow(...).when(spy)...`,
-`Mockito.doNothing().when(spy)...`, and similar methods.
-====
-
-[NOTE]
-====
 Only _singleton_ beans can be overridden. Any attempt to override a non-singleton bean
 will result in an exception.
 

framework-docs/modules/ROOT/pages/testing/mockmvc/htmlunit/webdriver.adoc

@@ -261,7 +261,7 @@ Kotlin::
 
 This improves on the design of our xref:testing/mockmvc/htmlunit/mah.adoc#spring-mvc-test-server-htmlunit-mah-usage[HtmlUnit test]
 by leveraging the Page Object Pattern. As we mentioned in
-xref:testing/mockmvc/htmlunit/webdriver.adoc#mockmvc-server-htmlunit-webdriver-why[Why WebDriver and MockMvc?], we can use the Page Object Pattern
+xref:testing/mockmvc/htmlunit/webdriver.adoc#spring-mvc-test-server-htmlunit-webdriver-why[Why WebDriver and MockMvc?], we can use the Page Object Pattern
 with HtmlUnit, but it is much easier with WebDriver. Consider the following
 `CreateMessagePage` implementation:
 

framework-docs/modules/ROOT/pages/testing/testcontext-framework/application-events.adoc

@@ -2,10 +2,10 @@
 = Application Events
 
 The TestContext framework provides support for recording
-xref:core/beans/context-introduction.adoc#context-functionality-events[application events]
-published in the `ApplicationContext` so that assertions can be performed against those
-events within tests. All events published during the execution of a single test are made
-available via the `ApplicationEvents` API which allows you to process the events as a
+xref:core/beans/context-introduction.adoc#context-functionality-events[application events] published in the
+`ApplicationContext` so that assertions can be performed against those events within
+tests. All events published during the execution of a single test are made available via
+the `ApplicationEvents` API which allows you to process the events as a
 `java.util.Stream`.
 
 To use `ApplicationEvents` in your tests, do the following.
@@ -16,23 +16,16 @@ To use `ApplicationEvents` in your tests, do the following.
   that `ApplicationEventsTestExecutionListener` is registered by default and only needs
   to be manually registered if you have custom configuration via
   `@TestExecutionListeners` that does not include the default listeners.
-* When using the
-  xref:testing/testcontext-framework/support-classes.adoc#testcontext-junit-jupiter-extension[SpringExtension for JUnit Jupiter],
-  declare a method parameter of type `ApplicationEvents` in a `@Test`, `@BeforeEach`, or
-  `@AfterEach` method.
-** Since `ApplicationEvents` is scoped to the lifecycle of the current test method, this
-   is the recommended approach.
-* Alternatively, you can annotate a field of type `ApplicationEvents` with `@Autowired`
-  and use that instance of `ApplicationEvents` in your test and lifecycle methods.
-
-NOTE: `ApplicationEvents` is registered with the `ApplicationContext` as a _resolvable
-dependency_ which is scoped to the lifecycle of the current test method. Consequently,
-`ApplicationEvents` cannot be accessed outside the lifecycle of a test method and cannot be
-`@Autowired` into the constructor of a test class.
+* Annotate a field of type `ApplicationEvents` with `@Autowired` and use that instance of
+  `ApplicationEvents` in your test and lifecycle methods (such as `@BeforeEach` and
+  `@AfterEach` methods in JUnit Jupiter).
+** When using the xref:testing/testcontext-framework/support-classes.adoc#testcontext-junit-jupiter-extension[SpringExtension for JUnit Jupiter], you may declare a method
+   parameter of type `ApplicationEvents` in a test or lifecycle method as an alternative
+   to an `@Autowired` field in the test class.
 
 The following test class uses the `SpringExtension` for JUnit Jupiter and
-{assertj-docs}[AssertJ] to assert the types of application events published while
-invoking a method in a Spring-managed component:
+{assertj-docs}[AssertJ] to assert the types of application events
+published while invoking a method in a Spring-managed component:
 
 // Don't use "quotes" in the "subs" section because of the asterisks in /* ... */
 [tabs]
@@ -45,10 +38,16 @@ Java::
 	@RecordApplicationEvents // <1>
 	class OrderServiceTests {
 
+		@Autowired
+		OrderService orderService;
+
+		@Autowired
+		ApplicationEvents events; // <2>
+
 		@Test
-		void submitOrder(@Autowired OrderService service, ApplicationEvents events) { // <2>
+		void submitOrder() {
 			// Invoke method in OrderService that publishes an event
-			service.submitOrder(new Order(/* ... */));
+			orderService.submitOrder(new Order(/* ... */));
 			// Verify that an OrderSubmitted event was published
 			long numEvents = events.stream(OrderSubmitted.class).count(); // <3>
 			assertThat(numEvents).isEqualTo(1);
@@ -67,10 +66,16 @@ Kotlin::
 	@RecordApplicationEvents // <1>
 	class OrderServiceTests {
 
+		@Autowired
+		lateinit var orderService: OrderService
+
+		@Autowired
+		lateinit var events: ApplicationEvents // <2>
+
 		@Test
-		fun submitOrder(@Autowired service: OrderService, events: ApplicationEvents) { // <2>
+		fun submitOrder() {
 			// Invoke method in OrderService that publishes an event
-			service.submitOrder(Order(/* ... */))
+			orderService.submitOrder(Order(/* ... */))
 			// Verify that an OrderSubmitted event was published
 			val numEvents = events.stream(OrderSubmitted::class).count() // <3>
 			assertThat(numEvents).isEqualTo(1)

framework-docs/modules/ROOT/pages/web/webflux-functional.adoc

@@ -294,28 +294,7 @@ allPartsEvents.windowUntil(PartEvent::isLast)
 ----
 ======
 
-NOTE: The body contents of the `PartEvent` objects must be completely consumed, relayed, or released to avoid memory leaks.
-
-The following shows how to bind request parameters, including an optional `DataBinder` customization:
-
-[tabs]
-======
-Java::
-+
-[source,java]
-----
-Pet pet = request.bind(Pet.class, dataBinder -> dataBinder.setAllowedFields("name"));
-----
-
-Kotlin::
-+
-[source,kotlin]
-----
-val pet = request.bind(Pet::class.java, {dataBinder -> dataBinder.setAllowedFields("name")})
-----
-======
-
-
+Note that the body contents of the `PartEvent` objects must be completely consumed, relayed, or released to avoid memory leaks.
 
 [[webflux-fn-response]]
 === ServerResponse

framework-docs/modules/ROOT/pages/web/webflux-webclient.adoc

@@ -2,18 +2,18 @@
 = WebClient
 :page-section-summary-toc: 1
 
-Spring WebFlux includes a client to perform HTTP requests. `WebClient` has a
-functional, fluent API based on Reactor (see xref:web/webflux-reactive-libraries.adoc[Reactive Libraries])
+Spring WebFlux includes a client to perform HTTP requests with. `WebClient` has a
+functional, fluent API based on Reactor, see xref:web-reactive.adoc#webflux-reactive-libraries[Reactive Libraries],
 which enables declarative composition of asynchronous logic without the need to deal with
-threads or concurrency. It is fully non-blocking, supports streaming, and relies on
+threads or concurrency. It is fully non-blocking, it supports streaming, and relies on
 the same xref:web/webflux/reactive-spring.adoc#webflux-codecs[codecs] that are also used to encode and
 decode request and response content on the server side.
 
-`WebClient` needs an HTTP client library to perform requests. There is built-in
+`WebClient` needs an HTTP client library to perform requests with. There is built-in
 support for the following:
 
 * {reactor-github-org}/reactor-netty[Reactor Netty]
 * {java-api}/java.net.http/java/net/http/HttpClient.html[JDK HttpClient]
 * https://github.com/jetty-project/jetty-reactive-httpclient[Jetty Reactive HttpClient]
 * https://hc.apache.org/index.html[Apache HttpComponents]
-* Others can be plugged in via `ClientHttpConnector`.
+* Others can be plugged via `ClientHttpConnector`.

framework-docs/modules/ROOT/pages/web/webflux-websocket.adoc

@@ -310,7 +310,7 @@ Java::
 			Flux<String> source = ... ;
 			Mono<Void> output = session.send(source.map(session::textMessage));	<2>
 
-			return input.and(output);											<3>
+			return Mono.zip(input, output).then();								<3>
 		}
 	}
 ----
@@ -338,7 +338,7 @@ Kotlin::
 			val source: Flux<String> = ...
 			val output = session.send(source.map(session::textMessage))		// <2>
 
-			return input.and(output)										// <3>
+			return Mono.zip(input, output).then()							// <3>
 		}
 	}
 ----

framework-docs/modules/ROOT/pages/web/webflux/controller/ann-requestmapping.adoc

@@ -112,11 +112,10 @@ You can map requests by using glob patterns and wildcards:
 | `+{name}+`
 | Matches a path segment and captures it as a variable named "name"
 | `+"/projects/{project}/versions"+` matches `+"/projects/spring/versions"+` and captures `+project=spring+`
-`+"/projects/{project}/versions"+` does not match `+"/projects/spring/framework/versions"+` as it captures a single path segment.
 
-| `{name:[a-z]+}`
-| Matches the regexp `[a-z]+` as a path variable named "name"
-| `/projects/{project:[a-z]+}/versions` matches `/projects/spring/versions` but not `/projects/spring1/versions`
+| `+{name:[a-z]+}+`
+| Matches the regexp `+"[a-z]+"+` as a path variable named "name"
+| `+"/projects/{project:[a-z]+}/versions"+` matches `+"/projects/spring/versions"+` but not `+"/projects/spring1/versions"+`
 
 | `+{*path}+`
 | Matches zero or more path segments until the end of the path and captures it as a variable named "path"
@@ -235,13 +234,10 @@ Kotlin::
 ======
 --
 
-URI path patterns can also have:
-
-- Embedded `${...}` placeholders that are resolved on startup via
-`PropertySourcesPlaceholderConfigurer` against local, system, environment, and
-other property sources. This is useful, for example, to parameterize a base URL based on
-external configuration.
-- SpEL expressions `#{...}`.
+URI path patterns can also have embedded `${...}` placeholders that are resolved on startup
+by using `PropertySourcesPlaceholderConfigurer` against local, system, environment, and
+other property sources. You can use this, for example, to parameterize a base URL based on
+some external configuration.
 
 NOTE: Spring WebFlux uses `PathPattern` and the `PathPatternParser` for URI path matching support.
 Both classes are located in `spring-web` and are expressly designed for use with HTTP URL

framework-docs/modules/ROOT/pages/web/webmvc-client.adoc

@@ -15,27 +15,27 @@ See xref:integration/rest-clients.adoc#rest-restclient[`RestClient`] for more de
 [[webmvc-webclient]]
 == `WebClient`
 
-`WebClient` is a reactive client for making HTTP requests with a fluent API.
+`WebClient` is a reactive client to perform HTTP requests with a fluent API.
 
-See xref:web/webflux-webclient.adoc[`WebClient`] for more details.
+See xref:web/webflux-webclient.adoc[WebClient] for more details.
 
 
 [[webmvc-resttemplate]]
 == `RestTemplate`
 
-`RestTemplate` is a synchronous client for making HTTP requests. It is the original
+`RestTemplate` is a synchronous client to perform HTTP requests. It is the original
 Spring REST client and exposes a simple, template-method API over underlying HTTP client
 libraries.
 
-See xref:integration/rest-clients.adoc#rest-resttemplate[`RestTemplate`] for details.
+See xref:integration/rest-clients.adoc[REST Endpoints] for details.
 
 
 [[webmvc-http-interface]]
 == HTTP Interface
 
-The Spring Framework lets you define an HTTP service as a Java interface with HTTP
+The Spring Frameworks lets you define an HTTP service as a Java interface with HTTP
 exchange methods. You can then generate a proxy that implements this interface and
 performs the exchanges. This helps to simplify HTTP remote access and provides additional
-flexibility for choosing an API style such as synchronous or reactive.
+flexibility for to choose an API style such as synchronous or reactive.
 
-See xref:integration/rest-clients.adoc#rest-http-interface[HTTP Interface] for details.
+See xref:integration/rest-clients.adoc#rest-http-interface[REST Endpoints] for details.

framework-docs/modules/ROOT/pages/web/webmvc-functional.adoc

@@ -184,26 +184,6 @@ val map = request.params()
 ----
 ======
 
-The following shows how to bind request parameters, including an optional `DataBinder` customization:
-
-[tabs]
-======
-Java::
-+
-[source,java]
-----
-Pet pet = request.bind(Pet.class, dataBinder -> dataBinder.setAllowedFields("name"));
-----
-
-Kotlin::
-+
-[source,kotlin]
-----
-val pet = request.bind(Pet::class.java, {dataBinder -> dataBinder.setAllowedFields("name")})
-----
-======
-
-
 [[webmvc-fn-response]]
 === ServerResponse
 

framework-docs/modules/ROOT/pages/web/webmvc/mvc-controller/ann-requestmapping.adoc

@@ -217,13 +217,10 @@ Kotlin::
 ----
 ======
 
-URI path patterns can also have:
-
-- Embedded `${...}` placeholders that are resolved on startup via
-`PropertySourcesPlaceholderConfigurer` against local, system, environment, and
-other property sources. This is useful, for example, to parameterize a base URL based on
-external configuration.
-- SpEL expression `#{...}`.
+URI path patterns can also have embedded `${...}` placeholders that are resolved on startup
+by using `PropertySourcesPlaceholderConfigurer` against local, system, environment, and
+other property sources. You can use this, for example, to parameterize a base URL based on
+some external configuration.
 
 
 [[mvc-ann-requestmapping-pattern-comparison]]

framework-platform/framework-platform.gradle

@@ -7,31 +7,31 @@ javaPlatform {
 }
 
 dependencies {
-	api(platform("com.fasterxml.jackson:jackson-bom:2.18.4.1"))
-	api(platform("io.micrometer:micrometer-bom:1.14.12"))
-	api(platform("io.netty:netty-bom:4.1.128.Final"))
+	api(platform("com.fasterxml.jackson:jackson-bom:2.18.4"))
+	api(platform("io.micrometer:micrometer-bom:1.14.9"))
+	api(platform("io.netty:netty-bom:4.1.123.Final"))
 	api(platform("io.netty:netty5-bom:5.0.0.Alpha5"))
-	api(platform("io.projectreactor:reactor-bom:2024.0.11"))
+	api(platform("io.projectreactor:reactor-bom:2024.0.8"))
 	api(platform("io.rsocket:rsocket-bom:1.1.5"))
-	api(platform("org.apache.groovy:groovy-bom:4.0.28"))
+	api(platform("org.apache.groovy:groovy-bom:4.0.27"))
 	api(platform("org.apache.logging.log4j:log4j-bom:2.21.1"))
-	api(platform("org.assertj:assertj-bom:3.27.6"))
-	api(platform("org.eclipse.jetty:jetty-bom:12.0.28"))
-	api(platform("org.eclipse.jetty.ee10:jetty-ee10-bom:12.0.28"))
+	api(platform("org.assertj:assertj-bom:3.27.3"))
+	api(platform("org.eclipse.jetty:jetty-bom:12.0.23"))
+	api(platform("org.eclipse.jetty.ee10:jetty-ee10-bom:12.0.23"))
 	api(platform("org.jetbrains.kotlinx:kotlinx-coroutines-bom:1.8.1"))
 	api(platform("org.jetbrains.kotlinx:kotlinx-serialization-bom:1.6.3"))
-	api(platform("org.junit:junit-bom:5.14.0"))
-	api(platform("org.mockito:mockito-bom:5.20.0"))
+	api(platform("org.junit:junit-bom:5.13.3"))
+	api(platform("org.mockito:mockito-bom:5.18.0"))
 
 	constraints {
-		api("com.fasterxml:aalto-xml:1.3.4")
+		api("com.fasterxml:aalto-xml:1.3.2")
 		api("com.fasterxml.woodstox:woodstox-core:6.7.0")
-		api("com.github.ben-manes.caffeine:caffeine:3.2.2")
+		api("com.github.ben-manes.caffeine:caffeine:3.2.1")
 		api("com.github.librepdf:openpdf:1.3.43")
 		api("com.google.code.findbugs:findbugs:3.0.1")
 		api("com.google.code.findbugs:jsr305:3.0.2")
-		api("com.google.code.gson:gson:2.13.2")
-		api("com.google.protobuf:protobuf-java-util:4.32.1")
+		api("com.google.code.gson:gson:2.13.1")
+		api("com.google.protobuf:protobuf-java-util:4.30.2")
 		api("com.h2database:h2:2.3.232")
 		api("com.jayway.jsonpath:json-path:2.9.0")
 		api("com.oracle.database.jdbc:ojdbc11:21.9.0.0")
@@ -53,11 +53,11 @@ dependencies {
 		api("io.r2dbc:r2dbc-h2:1.0.0.RELEASE")
 		api("io.r2dbc:r2dbc-spi-test:1.0.0.RELEASE")
 		api("io.r2dbc:r2dbc-spi:1.0.0.RELEASE")
-		api("io.reactivex.rxjava3:rxjava:3.1.12")
+		api("io.reactivex.rxjava3:rxjava:3.1.10")
 		api("io.smallrye.reactive:mutiny:1.10.0")
-		api("io.undertow:undertow-core:2.3.20.Final")
-		api("io.undertow:undertow-servlet:2.3.20.Final")
-		api("io.undertow:undertow-websockets-jsr:2.3.20.Final")
+		api("io.undertow:undertow-core:2.3.18.Final")
+		api("io.undertow:undertow-servlet:2.3.18.Final")
+		api("io.undertow:undertow-websockets-jsr:2.3.18.Final")
 		api("io.vavr:vavr:0.10.4")
 		api("jakarta.activation:jakarta.activation-api:2.0.1")
 		api("jakarta.annotation:jakarta.annotation-api:2.0.0")
@@ -90,17 +90,17 @@ dependencies {
 		api("junit:junit:4.13.2")
 		api("net.sf.jopt-simple:jopt-simple:5.0.4")
 		api("org.apache-extras.beanshell:bsh:2.0b6")
-        api("org.apache.activemq:activemq-broker:5.17.7")
-        api("org.apache.activemq:activemq-kahadb-store:5.17.7")
-        api("org.apache.activemq:activemq-stomp:5.17.7")
-        api("org.apache.activemq:artemis-jakarta-client:2.42.0")
-        api("org.apache.activemq:artemis-junit-5:2.42.0")
+		api("org.apache.activemq:activemq-broker:5.17.6")
+		api("org.apache.activemq:activemq-kahadb-store:5.17.6")
+		api("org.apache.activemq:activemq-stomp:5.17.6")
+		api("org.apache.activemq:artemis-jakarta-client:2.31.2")
+		api("org.apache.activemq:artemis-junit-5:2.31.2")
 		api("org.apache.commons:commons-pool2:2.9.0")
 		api("org.apache.derby:derby:10.16.1.1")
 		api("org.apache.derby:derbyclient:10.16.1.1")
 		api("org.apache.derby:derbytools:10.16.1.1")
 		api("org.apache.httpcomponents.client5:httpclient5:5.5")
-		api("org.apache.httpcomponents.core5:httpcore5-reactive:5.3.5")
+		api("org.apache.httpcomponents.core5:httpcore5-reactive:5.3.4")
 		api("org.apache.poi:poi-ooxml:5.2.5")
 		api("org.apache.tomcat.embed:tomcat-embed-core:10.1.28")
 		api("org.apache.tomcat.embed:tomcat-embed-websocket:10.1.28")
@@ -113,9 +113,9 @@ dependencies {
 		api("org.bouncycastle:bcpkix-jdk18on:1.72")
 		api("org.codehaus.jettison:jettison:1.5.4")
 		api("org.crac:crac:1.4.0")
-		api("org.dom4j:dom4j:2.2.0")
-		api("org.easymock:easymock:5.6.0")
-		api("org.eclipse.jetty:jetty-reactive-httpclient:4.0.12")
+		api("org.dom4j:dom4j:2.1.4")
+		api("org.easymock:easymock:5.5.0")
+		api("org.eclipse.jetty:jetty-reactive-httpclient:4.0.9")
 		api("org.eclipse.persistence:org.eclipse.persistence.jpa:3.0.4")
 		api("org.eclipse:yasson:2.0.4")
 		api("org.ehcache:ehcache:3.10.8")
@@ -129,24 +129,24 @@ dependencies {
 		api("org.hibernate:hibernate-core-jakarta:5.6.15.Final")
 		api("org.hibernate:hibernate-validator:7.0.5.Final")
 		api("org.hsqldb:hsqldb:2.7.4")
-		api("org.htmlunit:htmlunit:4.17.0")
+		api("org.htmlunit:htmlunit:4.13.0")
 		api("org.javamoney:moneta:1.4.4")
-		api("org.jruby:jruby:9.4.13.0")
+		api("org.jruby:jruby:9.4.12.0")
 		api("org.junit.support:testng-engine:1.0.5")
 		api("org.mozilla:rhino:1.7.15")
 		api("org.ogce:xpp3:1.1.6")
 		api("org.python:jython-standalone:2.7.4")
 		api("org.quartz-scheduler:quartz:2.3.2")
-		api("org.seleniumhq.selenium:htmlunit3-driver:4.36.1")
-		api("org.seleniumhq.selenium:selenium-java:4.36.0")
+		api("org.seleniumhq.selenium:htmlunit3-driver:4.33.0")
+		api("org.seleniumhq.selenium:selenium-java:4.34.0")
 		api("org.skyscreamer:jsonassert:1.5.3")
 		api("org.slf4j:slf4j-api:2.0.17")
 		api("org.testng:testng:7.11.0")
 		api("org.webjars:underscorejs:1.8.3")
 		api("org.webjars:webjars-locator-core:0.59")
 		api("org.webjars:webjars-locator-lite:1.1.0")
-		api("org.xmlunit:xmlunit-assertj:2.10.4")
-		api("org.xmlunit:xmlunit-matchers:2.10.4")
-		api("org.yaml:snakeyaml:2.5")
+		api("org.xmlunit:xmlunit-assertj:2.10.3")
+		api("org.xmlunit:xmlunit-matchers:2.10.3")
+		api("org.yaml:snakeyaml:2.4")
 	}
 }

gradle.properties

@@ -1,4 +1,4 @@
-version=6.2.12
+version=6.2.9
 
 org.gradle.caching=true
 org.gradle.jvmargs=-Xmx2048m

spring-aop/src/main/java/org/springframework/aop/config/AopConfigUtils.java

@@ -121,12 +121,12 @@ public abstract class AopConfigUtils {
 		Assert.notNull(registry, "BeanDefinitionRegistry must not be null");
 
 		if (registry.containsBeanDefinition(AUTO_PROXY_CREATOR_BEAN_NAME)) {
-			BeanDefinition beanDefinition = registry.getBeanDefinition(AUTO_PROXY_CREATOR_BEAN_NAME);
-			if (!cls.getName().equals(beanDefinition.getBeanClassName())) {
-				int currentPriority = findPriorityForClass(beanDefinition.getBeanClassName());
+			BeanDefinition apcDefinition = registry.getBeanDefinition(AUTO_PROXY_CREATOR_BEAN_NAME);
+			if (!cls.getName().equals(apcDefinition.getBeanClassName())) {
+				int currentPriority = findPriorityForClass(apcDefinition.getBeanClassName());
 				int requiredPriority = findPriorityForClass(cls);
 				if (currentPriority < requiredPriority) {
-					beanDefinition.setBeanClassName(cls.getName());
+					apcDefinition.setBeanClassName(cls.getName());
 				}
 			}
 			return null;
@@ -134,8 +134,8 @@ public abstract class AopConfigUtils {
 
 		RootBeanDefinition beanDefinition = new RootBeanDefinition(cls);
 		beanDefinition.setSource(source);
-		beanDefinition.setRole(BeanDefinition.ROLE_INFRASTRUCTURE);
 		beanDefinition.getPropertyValues().add("order", Ordered.HIGHEST_PRECEDENCE);
+		beanDefinition.setRole(BeanDefinition.ROLE_INFRASTRUCTURE);
 		registry.registerBeanDefinition(AUTO_PROXY_CREATOR_BEAN_NAME, beanDefinition);
 		return beanDefinition;
 	}

spring-aop/src/main/java/org/springframework/aop/framework/AbstractAdvisingBeanPostProcessor.java

@@ -117,7 +117,6 @@ public abstract class AbstractAdvisingBeanPostProcessor extends ProxyProcessorSu
 			}
 			proxyFactory.addAdvisor(this.advisor);
 			customizeProxyFactory(proxyFactory);
-			proxyFactory.setPreFiltered(true);
 
 			// Use original ClassLoader if bean class not locally loaded in overriding class loader
 			ClassLoader classLoader = getProxyClassLoader();

spring-aop/src/main/java/org/springframework/aop/framework/autoproxy/AbstractBeanFactoryAwareAdvisingPostProcessor.java

@@ -24,9 +24,9 @@ import org.springframework.beans.factory.config.ConfigurableListableBeanFactory;
 import org.springframework.lang.Nullable;
 
 /**
- * Extension of {@link AbstractAdvisingBeanPostProcessor} which implements
- * {@link BeanFactoryAware}, adds exposure of the original target class for each
- * proxied bean ({@link AutoProxyUtils#ORIGINAL_TARGET_CLASS_ATTRIBUTE}),
+ * Extension of {@link AbstractAutoProxyCreator} which implements {@link BeanFactoryAware},
+ * adds exposure of the original target class for each proxied bean
+ * ({@link AutoProxyUtils#ORIGINAL_TARGET_CLASS_ATTRIBUTE}),
  * and participates in an externally enforced target-class mode for any given bean
  * ({@link AutoProxyUtils#PRESERVE_TARGET_CLASS_ATTRIBUTE}).
  * This post-processor is therefore aligned with {@link AbstractAutoProxyCreator}.

spring-aop/src/main/java/org/springframework/aop/scope/ScopedProxyUtils.java

@@ -81,19 +81,13 @@ public abstract class ScopedProxyUtils {
 		// Copy autowire settings from original bean definition.
 		proxyDefinition.setAutowireCandidate(targetDefinition.isAutowireCandidate());
 		proxyDefinition.setPrimary(targetDefinition.isPrimary());
-		proxyDefinition.setFallback(targetDefinition.isFallback());
 		if (targetDefinition instanceof AbstractBeanDefinition abd) {
-			proxyDefinition.setDefaultCandidate(abd.isDefaultCandidate());
 			proxyDefinition.copyQualifiersFrom(abd);
 		}
 
 		// The target bean should be ignored in favor of the scoped proxy.
 		targetDefinition.setAutowireCandidate(false);
 		targetDefinition.setPrimary(false);
-		targetDefinition.setFallback(false);
-		if (targetDefinition instanceof AbstractBeanDefinition abd) {
-			abd.setDefaultCandidate(false);
-		}
 
 		// Register the target bean as separate bean in the factory.
 		registry.registerBeanDefinition(targetBeanName, targetDefinition);

spring-aop/src/test/java/org/springframework/aop/interceptor/AsyncExecutionInterceptorTests.java

@@ -37,6 +37,7 @@ import static org.mockito.Mockito.verify;
  * Tests for {@link AsyncExecutionInterceptor}.
  *
  * @author Bao Ngo
+ * @since 7.0
  */
 class AsyncExecutionInterceptorTests {
 
@@ -61,13 +62,11 @@ class AsyncExecutionInterceptorTests {
 		O run();
 	}
 
-
 	static class FutureRunner implements GenericRunner<Future<Void>> {
-
 		@Override
 		public Future<Void> run() {
-			return CompletableFuture.runAsync(() -> {});
+			return CompletableFuture.runAsync(() -> {
+			});
 		}
 	}
-
 }

spring-aop/src/test/java/org/springframework/aop/scope/ScopedProxyUtilsTests.java

@@ -18,15 +18,6 @@ package org.springframework.aop.scope;
 
 import org.junit.jupiter.api.Test;
 
-import org.springframework.beans.factory.config.BeanDefinition;
-import org.springframework.beans.factory.config.BeanDefinitionHolder;
-import org.springframework.beans.factory.support.AbstractBeanDefinition;
-import org.springframework.beans.factory.support.AutowireCandidateQualifier;
-import org.springframework.beans.factory.support.BeanDefinitionRegistry;
-import org.springframework.beans.factory.support.GenericBeanDefinition;
-import org.springframework.beans.factory.support.RootBeanDefinition;
-import org.springframework.beans.factory.support.SimpleBeanDefinitionRegistry;
-
 import static org.assertj.core.api.Assertions.assertThat;
 import static org.assertj.core.api.Assertions.assertThatIllegalArgumentException;
 
@@ -34,7 +25,6 @@ import static org.assertj.core.api.Assertions.assertThatIllegalArgumentException
  * Tests for {@link ScopedProxyUtils}.
  *
  * @author Sam Brannen
- * @author Juergen Hoeller
  * @since 5.1.10
  */
 class ScopedProxyUtilsTests {
@@ -63,79 +53,15 @@ class ScopedProxyUtilsTests {
 	@Test
 	void getOriginalBeanNameForNullTargetBean() {
 		assertThatIllegalArgumentException()
-				.isThrownBy(() -> ScopedProxyUtils.getOriginalBeanName(null))
-				.withMessage("bean name 'null' does not refer to the target of a scoped proxy");
+			.isThrownBy(() -> ScopedProxyUtils.getOriginalBeanName(null))
+			.withMessage("bean name 'null' does not refer to the target of a scoped proxy");
 	}
 
 	@Test
 	void getOriginalBeanNameForNonScopedTarget() {
 		assertThatIllegalArgumentException()
-				.isThrownBy(() -> ScopedProxyUtils.getOriginalBeanName("myBean"))
-				.withMessage("bean name 'myBean' does not refer to the target of a scoped proxy");
-	}
-
-	@Test
-	void createScopedProxyTargetAppliesAutowireSettingsToProxyBeanDefinition() {
-		AbstractBeanDefinition targetDefinition = new GenericBeanDefinition();
-		// Opposite of defaults
-		targetDefinition.setAutowireCandidate(false);
-		targetDefinition.setDefaultCandidate(false);
-		targetDefinition.setPrimary(true);
-		targetDefinition.setFallback(true);
-
-		BeanDefinitionRegistry registry = new SimpleBeanDefinitionRegistry();
-		BeanDefinitionHolder proxyHolder = ScopedProxyUtils.createScopedProxy(
-				new BeanDefinitionHolder(targetDefinition, "myBean"), registry, false);
-		AbstractBeanDefinition proxyBeanDefinition = (AbstractBeanDefinition) proxyHolder.getBeanDefinition();
-
-		assertThat(proxyBeanDefinition.isAutowireCandidate()).isFalse();
-		assertThat(proxyBeanDefinition.isDefaultCandidate()).isFalse();
-		assertThat(proxyBeanDefinition.isPrimary()).isTrue();
-		assertThat(proxyBeanDefinition.isFallback()).isTrue();
-	}
-
-	@Test
-	void createScopedProxyTargetAppliesBeanAttributesToProxyBeanDefinition() {
-		GenericBeanDefinition targetDefinition = new GenericBeanDefinition();
-		// Opposite of defaults
-		targetDefinition.setRole(BeanDefinition.ROLE_INFRASTRUCTURE);
-		targetDefinition.setSource("theSource");
-		targetDefinition.addQualifier(new AutowireCandidateQualifier("myQualifier"));
-
-		BeanDefinitionRegistry registry = new SimpleBeanDefinitionRegistry();
-		BeanDefinitionHolder proxyHolder = ScopedProxyUtils.createScopedProxy(
-				new BeanDefinitionHolder(targetDefinition, "myBean"), registry, false);
-		BeanDefinition proxyBeanDefinition = proxyHolder.getBeanDefinition();
-
-		assertThat(proxyBeanDefinition.getRole()).isEqualTo(BeanDefinition.ROLE_INFRASTRUCTURE);
-		assertThat(proxyBeanDefinition).isInstanceOf(RootBeanDefinition.class);
-		assertThat(proxyBeanDefinition.getPropertyValues()).hasSize(2);
-		assertThat(proxyBeanDefinition.getPropertyValues().get("proxyTargetClass")).isEqualTo(false);
-		assertThat(proxyBeanDefinition.getPropertyValues().get("targetBeanName")).isEqualTo(
-				ScopedProxyUtils.getTargetBeanName("myBean"));
-
-		RootBeanDefinition rootBeanDefinition = (RootBeanDefinition) proxyBeanDefinition;
-		assertThat(rootBeanDefinition.getQualifiers()).hasSize(1);
-		assertThat(rootBeanDefinition.hasQualifier("myQualifier")).isTrue();
-		assertThat(rootBeanDefinition.getSource()).isEqualTo("theSource");
-	}
-
-	@Test
-	void createScopedProxyTargetCleansAutowireSettingsInTargetDefinition() {
-		AbstractBeanDefinition targetDefinition = new GenericBeanDefinition();
-		targetDefinition.setAutowireCandidate(true);
-		targetDefinition.setDefaultCandidate(true);
-		targetDefinition.setPrimary(true);
-		targetDefinition.setFallback(true);
-
-		BeanDefinitionRegistry registry = new SimpleBeanDefinitionRegistry();
-		ScopedProxyUtils.createScopedProxy(
-				new BeanDefinitionHolder(targetDefinition, "myBean"), registry, false);
-
-		assertThat(targetDefinition.isAutowireCandidate()).isFalse();
-		assertThat(targetDefinition.isDefaultCandidate()).isFalse();
-		assertThat(targetDefinition.isPrimary()).isFalse();
-		assertThat(targetDefinition.isFallback()).isFalse();
+			.isThrownBy(() -> ScopedProxyUtils.getOriginalBeanName("myBean"))
+			.withMessage("bean name 'myBean' does not refer to the target of a scoped proxy");
 	}
 
 }

spring-aop/src/test/kotlin/org/springframework/aop/framework/CglibAopProxyKotlinTests.kt

@@ -19,7 +19,6 @@ package org.springframework.aop.framework
 import org.assertj.core.api.Assertions.assertThat
 import org.assertj.core.api.Assertions.assertThatThrownBy
 import org.junit.jupiter.api.Test
-import java.time.LocalDateTime
 
 /**
  * Tests for Kotlin support in [CglibAopProxy].
@@ -49,13 +48,6 @@ class CglibAopProxyKotlinTests {
 		assertThatThrownBy { proxy.checkedException() }.isInstanceOf(CheckedException::class.java)
 	}
 
-	@Test // gh-35487
-	fun jvmDefault() {
-		val proxyFactory = ProxyFactory()
-		proxyFactory.setTarget(AddressRepo())
-		proxyFactory.proxy
-	}
-
 
 	open class MyKotlinBean {
 
@@ -71,24 +63,4 @@ class CglibAopProxyKotlinTests {
 	}
 
 	class CheckedException() : Exception()
-
-	open class AddressRepo(): CrudRepo<Address, Int>
-
-	interface CrudRepo<E : Any, ID : Any> {
-		fun save(e: E): E {
-			return e
-		}
-		fun delete(id: ID): Long {
-			return 0L
-		}
-	}
-
-	data class Address(
-		val id: Int = 0,
-		val street: String,
-		val version: Int = 0,
-		val createdAt: LocalDateTime? = null,
-		val updatedAt: LocalDateTime? = null,
-	)
-
 }

spring-beans/src/main/java/org/springframework/beans/factory/aot/InstanceSupplierCodeGenerator.java

@@ -294,12 +294,9 @@ public class InstanceSupplierCodeGenerator {
 
 		this.generationContext.getRuntimeHints().reflection().registerMethod(factoryMethod, ExecutableMode.INVOKE);
 		GeneratedMethod getInstanceMethod = generateGetInstanceSupplierMethod(method -> {
-			CodeWarnings codeWarnings = new CodeWarnings();
 			Class<?> suppliedType = ClassUtils.resolvePrimitiveIfNecessary(factoryMethod.getReturnType());
-			codeWarnings.detectDeprecation(suppliedType, factoryMethod);
 			method.addJavadoc("Get the bean instance supplier for '$L'.", beanName);
 			method.addModifiers(PRIVATE_STATIC);
-			codeWarnings.suppress(method);
 			method.returns(ParameterizedTypeName.get(BeanInstanceSupplier.class, suppliedType));
 			method.addStatement(generateInstanceSupplierForFactoryMethod(
 					factoryMethod, suppliedType, targetClass, factoryMethod.getName()));

spring-beans/src/main/java/org/springframework/beans/factory/config/ConfigurableListableBeanFactory.java

@@ -152,18 +152,6 @@ public interface ConfigurableListableBeanFactory
 	 */
 	boolean isConfigurationFrozen();
 
-	/**
-	 * Mark current thread as main bootstrap thread for singleton instantiation,
-	 * with lenient bootstrap locking applying for background threads.
-	 * <p>Any such marker is to be removed at the end of the managed bootstrap in
-	 * {@link #preInstantiateSingletons()}.
-	 * @since 6.2.12
-	 * @see #setBootstrapExecutor
-	 * @see #preInstantiateSingletons()
-	 */
-	default void prepareSingletonBootstrap() {
-	}
-
 	/**
 	 * Ensure that all non-lazy-init singletons are instantiated, also considering
 	 * {@link org.springframework.beans.factory.FactoryBean FactoryBeans}.
@@ -171,7 +159,6 @@ public interface ConfigurableListableBeanFactory
 	 * @throws BeansException if one of the singleton beans could not be created.
 	 * Note: This may have left the factory with some beans already initialized!
 	 * Call {@link #destroySingletons()} for full cleanup in this case.
-	 * @see #prepareSingletonBootstrap()
 	 * @see #destroySingletons()
 	 */
 	void preInstantiateSingletons() throws BeansException;

spring-beans/src/main/java/org/springframework/beans/factory/support/AbstractBeanFactory.java

@@ -520,7 +520,8 @@ public abstract class AbstractBeanFactory extends FactoryBeanRegistrySupport imp
 	 * to check whether the bean with the given name matches the specified type. Allow
 	 * additional constraints to be applied to ensure that beans are not created early.
 	 * @param name the name of the bean to query
-	 * @param typeToMatch the type to match against (as a {@code ResolvableType})
+	 * @param typeToMatch the type to match against (as a
+	 * {@code ResolvableType})
 	 * @return {@code true} if the bean type matches, {@code false} if it
 	 * doesn't match or cannot be determined yet
 	 * @throws NoSuchBeanDefinitionException if there is no bean with the given name

spring-beans/src/main/java/org/springframework/beans/factory/support/DefaultListableBeanFactory.java

@@ -58,7 +58,6 @@ import org.springframework.beans.factory.BeanFactoryAware;
 import org.springframework.beans.factory.BeanFactoryUtils;
 import org.springframework.beans.factory.BeanNotOfRequiredTypeException;
 import org.springframework.beans.factory.CannotLoadBeanClassException;
-import org.springframework.beans.factory.FactoryBean;
 import org.springframework.beans.factory.InjectionPoint;
 import org.springframework.beans.factory.NoSuchBeanDefinitionException;
 import org.springframework.beans.factory.NoUniqueBeanDefinitionException;
@@ -197,8 +196,8 @@ public class DefaultListableBeanFactory extends AbstractAutowireCapableBeanFacto
 	/** Map from bean name to merged BeanDefinitionHolder. */
 	private final Map<String, BeanDefinitionHolder> mergedBeanDefinitionHolders = new ConcurrentHashMap<>(256);
 
-	/** Map of bean definition names with a primary marker plus corresponding type. */
-	private final Map<String, Class<?>> primaryBeanNamesWithType = new ConcurrentHashMap<>(16);
+	/** Set of bean definition names with a primary marker. */
+	private final Set<String> primaryBeanNames = ConcurrentHashMap.newKeySet(16);
 
 	/** Map of singleton and non-singleton bean names, keyed by dependency type. */
 	private final Map<Class<?>, String[]> allBeanNamesByType = new ConcurrentHashMap<>(64);
@@ -1038,7 +1037,7 @@ public class DefaultListableBeanFactory extends AbstractAutowireCapableBeanFacto
 	protected void cacheMergedBeanDefinition(RootBeanDefinition mbd, String beanName) {
 		super.cacheMergedBeanDefinition(mbd, beanName);
 		if (mbd.isPrimary()) {
-			this.primaryBeanNamesWithType.put(beanName, Void.class);
+			this.primaryBeanNames.add(beanName);
 		}
 	}
 
@@ -1102,11 +1101,6 @@ public class DefaultListableBeanFactory extends AbstractAutowireCapableBeanFacto
 		return null;
 	}
 
-	@Override
-	public void prepareSingletonBootstrap() {
-		this.mainThreadPrefix = getThreadNamePrefix();
-	}
-
 	@Override
 	public void preInstantiateSingletons() throws BeansException {
 		if (logger.isTraceEnabled()) {
@@ -1118,12 +1112,11 @@ public class DefaultListableBeanFactory extends AbstractAutowireCapableBeanFacto
 		List<String> beanNames = new ArrayList<>(this.beanDefinitionNames);
 
 		// Trigger initialization of all non-lazy singleton beans...
+		List<CompletableFuture<?>> futures = new ArrayList<>();
+
 		this.preInstantiationThread.set(PreInstantiation.MAIN);
-		if (this.mainThreadPrefix == null) {
-			this.mainThreadPrefix = getThreadNamePrefix();
-		}
+		this.mainThreadPrefix = getThreadNamePrefix();
 		try {
-			List<CompletableFuture<?>> futures = new ArrayList<>();
 			for (String beanName : beanNames) {
 				RootBeanDefinition mbd = getMergedLocalBeanDefinition(beanName);
 				if (!mbd.isAbstract() && mbd.isSingleton()) {
@@ -1133,20 +1126,21 @@ public class DefaultListableBeanFactory extends AbstractAutowireCapableBeanFacto
 					}
 				}
 			}
-			if (!futures.isEmpty()) {
-				try {
-					CompletableFuture.allOf(futures.toArray(new CompletableFuture<?>[0])).join();
-				}
-				catch (CompletionException ex) {
-					ReflectionUtils.rethrowRuntimeException(ex.getCause());
-				}
-			}
 		}
 		finally {
 			this.mainThreadPrefix = null;
 			this.preInstantiationThread.remove();
 		}
 
+		if (!futures.isEmpty()) {
+			try {
+				CompletableFuture.allOf(futures.toArray(new CompletableFuture<?>[0])).join();
+			}
+			catch (CompletionException ex) {
+				ReflectionUtils.rethrowRuntimeException(ex.getCause());
+			}
+		}
+
 		// Trigger post-initialization callback for all applicable beans...
 		for (String beanName : beanNames) {
 			Object singletonInstance = getSingleton(beanName, false);
@@ -1319,7 +1313,7 @@ public class DefaultListableBeanFactory extends AbstractAutowireCapableBeanFacto
 
 		// Cache a primary marker for the given bean.
 		if (beanDefinition.isPrimary()) {
-			this.primaryBeanNamesWithType.put(beanName, Void.class);
+			this.primaryBeanNames.add(beanName);
 		}
 	}
 
@@ -1411,7 +1405,7 @@ public class DefaultListableBeanFactory extends AbstractAutowireCapableBeanFacto
 		destroySingleton(beanName);
 
 		// Remove a cached primary marker for the given bean.
-		this.primaryBeanNamesWithType.remove(beanName);
+		this.primaryBeanNames.remove(beanName);
 
 		// Notify all post-processors that the specified bean definition has been reset.
 		for (MergedBeanDefinitionPostProcessor processor : getBeanPostProcessorCache().mergedDefinition) {
@@ -1461,30 +1455,11 @@ public class DefaultListableBeanFactory extends AbstractAutowireCapableBeanFacto
 		}
 	}
 
-	@Override
-	protected void addSingleton(String beanName, Object singletonObject) {
-		super.addSingleton(beanName, singletonObject);
-
-		Predicate<Class<?>> filter = (beanType -> beanType != Object.class && beanType.isInstance(singletonObject));
-		this.allBeanNamesByType.keySet().removeIf(filter);
-		this.singletonBeanNamesByType.keySet().removeIf(filter);
-
-		if (this.primaryBeanNamesWithType.containsKey(beanName) && singletonObject.getClass() != NullBean.class) {
-			Class<?> beanType = (singletonObject instanceof FactoryBean<?> fb ?
-					getTypeForFactoryBean(fb) : singletonObject.getClass());
-			if (beanType != null) {
-				this.primaryBeanNamesWithType.put(beanName, beanType);
-			}
-		}
-	}
-
 	@Override
 	public void registerSingleton(String beanName, Object singletonObject) throws IllegalStateException {
 		super.registerSingleton(beanName, singletonObject);
-
 		updateManualSingletonNames(set -> set.add(beanName), set -> !this.beanDefinitionMap.containsKey(beanName));
-		this.allBeanNamesByType.remove(Object.class);
-		this.singletonBeanNamesByType.remove(Object.class);
+		clearByTypeCache();
 	}
 
 	@Override
@@ -2108,9 +2083,8 @@ public class DefaultListableBeanFactory extends AbstractAutowireCapableBeanFacto
 					boolean candidateLocal = containsBeanDefinition(candidateBeanName);
 					boolean primaryLocal = containsBeanDefinition(primaryBeanName);
 					if (candidateLocal == primaryLocal) {
-						String message = "more than one 'primary' bean found among candidates: " + candidates.keySet();
-						logger.trace(message);
-						throw new NoUniqueBeanDefinitionException(requiredType, candidates.size(), message);
+						throw new NoUniqueBeanDefinitionException(requiredType, candidates.size(),
+								"more than one 'primary' bean found among candidates: " + candidates.keySet());
 					}
 					else if (candidateLocal) {
 						primaryBeanName = candidateBeanName;
@@ -2287,12 +2261,8 @@ public class DefaultListableBeanFactory extends AbstractAutowireCapableBeanFacto
 	 * not matching the given bean name.
 	 */
 	private boolean hasPrimaryConflict(String beanName, Class<?> dependencyType) {
-		for (Map.Entry<String, Class<?>> candidate : this.primaryBeanNamesWithType.entrySet()) {
-			String candidateName = candidate.getKey();
-			Class<?> candidateType = candidate.getValue();
-			if (!candidateName.equals(beanName) && (candidateType != Void.class ?
-					dependencyType.isAssignableFrom(candidateType) :  // cached singleton class for primary bean
-					isTypeMatch(candidateName, dependencyType))) {  // not instantiated yet or not a singleton
+		for (String candidate : this.primaryBeanNames) {
+			if (isTypeMatch(candidate, dependencyType) && !candidate.equals(beanName)) {
 				return true;
 			}
 		}

spring-beans/src/main/java/org/springframework/beans/factory/support/DisposableBeanAdapter.java

@@ -23,7 +23,6 @@ import java.util.ArrayList;
 import java.util.List;
 import java.util.concurrent.CountDownLatch;
 import java.util.concurrent.ExecutionException;
-import java.util.concurrent.ExecutorService;
 import java.util.concurrent.Future;
 
 import org.apache.commons.logging.Log;
@@ -419,29 +418,14 @@ class DisposableBeanAdapter implements DisposableBean, Runnable, Serializable {
 		String destroyMethodName = beanDefinition.resolvedDestroyMethodName;
 		if (destroyMethodName == null) {
 			destroyMethodName = beanDefinition.getDestroyMethodName();
-			boolean autoCloseable = AutoCloseable.class.isAssignableFrom(target);
-			boolean executorService = ExecutorService.class.isAssignableFrom(target);
+			boolean autoCloseable = (AutoCloseable.class.isAssignableFrom(target));
 			if (AbstractBeanDefinition.INFER_METHOD.equals(destroyMethodName) ||
-					(destroyMethodName == null && (autoCloseable || executorService))) {
+					(destroyMethodName == null && autoCloseable)) {
 				// Only perform destroy method inference in case of the bean
 				// not explicitly implementing the DisposableBean interface
 				destroyMethodName = null;
 				if (!(DisposableBean.class.isAssignableFrom(target))) {
-					if (executorService) {
-						destroyMethodName = SHUTDOWN_METHOD_NAME;
-						try {
-							// On JDK 19+, avoid the ExecutorService-level AutoCloseable default implementation
-							// which awaits task termination for 1 day, even for delayed tasks such as cron jobs.
-							// Custom close() implementations in ExecutorService subclasses are still accepted.
-							if (target.getMethod(CLOSE_METHOD_NAME).getDeclaringClass() != ExecutorService.class) {
-								destroyMethodName = CLOSE_METHOD_NAME;
-							}
-						}
-						catch (NoSuchMethodException ex) {
-							// Ignore - stick with shutdown()
-						}
-					}
-					else if (autoCloseable) {
+					if (autoCloseable) {
 						destroyMethodName = CLOSE_METHOD_NAME;
 					}
 					else {

spring-beans/src/main/java/org/springframework/beans/factory/support/FactoryBeanRegistrySupport.java

@@ -128,48 +128,43 @@ public abstract class FactoryBeanRegistrySupport extends DefaultSingletonBeanReg
 				locked = (lockFlag && this.singletonLock.tryLock());
 			}
 			try {
-				// Defensively synchronize against non-thread-safe FactoryBean.getObject() implementations,
-				// potentially to be called from a background thread while the main thread currently calls
-				// the same getObject() method within the singleton lock.
-				synchronized (factory) {
-					Object object = this.factoryBeanObjectCache.get(beanName);
-					if (object == null) {
-						object = doGetObjectFromFactoryBean(factory, beanName);
-						// Only post-process and store if not put there already during getObject() call above
-						// (for example, because of circular reference processing triggered by custom getBean calls)
-						Object alreadyThere = this.factoryBeanObjectCache.get(beanName);
-						if (alreadyThere != null) {
-							object = alreadyThere;
-						}
-						else {
-							if (shouldPostProcess) {
+				Object object = this.factoryBeanObjectCache.get(beanName);
+				if (object == null) {
+					object = doGetObjectFromFactoryBean(factory, beanName);
+					// Only post-process and store if not put there already during getObject() call above
+					// (for example, because of circular reference processing triggered by custom getBean calls)
+					Object alreadyThere = this.factoryBeanObjectCache.get(beanName);
+					if (alreadyThere != null) {
+						object = alreadyThere;
+					}
+					else {
+						if (shouldPostProcess) {
+							if (locked) {
+								if (isSingletonCurrentlyInCreation(beanName)) {
+									// Temporarily return non-post-processed object, not storing it yet
+									return object;
+								}
+								beforeSingletonCreation(beanName);
+							}
+							try {
+								object = postProcessObjectFromFactoryBean(object, beanName);
+							}
+							catch (Throwable ex) {
+								throw new BeanCreationException(beanName,
+										"Post-processing of FactoryBean's singleton object failed", ex);
+							}
+							finally {
 								if (locked) {
-									if (isSingletonCurrentlyInCreation(beanName)) {
-										// Temporarily return non-post-processed object, not storing it yet
-										return object;
-									}
-									beforeSingletonCreation(beanName);
-								}
-								try {
-									object = postProcessObjectFromFactoryBean(object, beanName);
-								}
-								catch (Throwable ex) {
-									throw new BeanCreationException(beanName,
-											"Post-processing of FactoryBean's singleton object failed", ex);
-								}
-								finally {
-									if (locked) {
-										afterSingletonCreation(beanName);
-									}
+									afterSingletonCreation(beanName);
 								}
 							}
-							if (containsSingleton(beanName)) {
-								this.factoryBeanObjectCache.put(beanName, object);
-							}
+						}
+						if (containsSingleton(beanName)) {
+							this.factoryBeanObjectCache.put(beanName, object);
 						}
 					}
-					return object;
 				}
+				return object;
 			}
 			finally {
 				if (locked) {

spring-beans/src/main/java/org/springframework/beans/propertyeditors/PathEditor.java

@@ -103,7 +103,7 @@ public class PathEditor extends PropertyEditorSupport {
 		if (resource == null) {
 			setValue(null);
 		}
-		else if (nioPathCandidate && (!resource.isFile() || !resource.exists())) {
+		else if (nioPathCandidate && !resource.exists()) {
 			setValue(Paths.get(text).normalize());
 		}
 		else {

spring-beans/src/test/java/org/springframework/beans/factory/DefaultListableBeanFactoryTests.java

@@ -87,7 +87,6 @@ import org.springframework.core.io.Resource;
 import org.springframework.core.io.UrlResource;
 import org.springframework.core.testfixture.io.SerializationTestUtils;
 import org.springframework.lang.Nullable;
-import org.springframework.util.StringUtils;
 import org.springframework.util.StringValueResolver;
 
 import static org.assertj.core.api.Assertions.assertThat;
@@ -1799,7 +1798,7 @@ class DefaultListableBeanFactoryTests {
 
 		assertThatExceptionOfType(NoUniqueBeanDefinitionException.class)
 				.isThrownBy(() -> lbf.getBean(TestBean.class))
-				.withMessageEndingWith("more than one 'primary' bean found among candidates: [bd1, bd2]");
+				.withMessageContaining("more than one 'primary'");
 	}
 
 	@Test
@@ -2123,7 +2122,7 @@ class DefaultListableBeanFactoryTests {
 
 		assertThatExceptionOfType(NoUniqueBeanDefinitionException.class)
 				.isThrownBy(() -> lbf.getBean(ConstructorDependency.class, 42))
-				.withMessageEndingWith("more than one 'primary' bean found among candidates: [bd1, bd2]");
+				.withMessageContaining("more than one 'primary'");
 	}
 
 	@Test
@@ -3203,33 +3202,6 @@ class DefaultListableBeanFactoryTests {
 		assertThat(holder.getNonPublicEnum()).isEqualTo(NonPublicEnum.VALUE_1);
 	}
 
-	@Test
-	void mostSpecificCacheEntryForTypeMatching() {
-		RootBeanDefinition bd1 = new RootBeanDefinition();
-		bd1.setFactoryBeanName("config");
-		bd1.setFactoryMethodName("create");
-		lbf.registerBeanDefinition("config", new RootBeanDefinition(BeanWithFactoryMethod.class));
-		lbf.registerBeanDefinition("bd1", bd1);
-		lbf.registerBeanDefinition("bd2", new RootBeanDefinition(NestedTestBean.class));
-		lbf.freezeConfiguration();
-
-		String[] allBeanNames = lbf.getBeanNamesForType(Object.class);
-		String[] nestedBeanNames = lbf.getBeanNamesForType(NestedTestBean.class);
-		assertThat(lbf.getType("bd1")).isEqualTo(TestBean.class);
-		assertThat(lbf.getBeanNamesForType(TestBean.class)).containsExactly("bd1");
-		assertThat(lbf.getBeanNamesForType(DerivedTestBean.class)).isEmpty();
-		lbf.getBean("bd1");
-		assertThat(lbf.getType("bd1")).isEqualTo(DerivedTestBean.class);
-		assertThat(lbf.getBeanNamesForType(TestBean.class)).containsExactly("bd1");
-		assertThat(lbf.getBeanNamesForType(DerivedTestBean.class)).containsExactly("bd1");
-		assertThat(lbf.getBeanNamesForType(NestedTestBean.class)).isSameAs(nestedBeanNames);
-		assertThat(lbf.getBeanNamesForType(Object.class)).isSameAs(allBeanNames);
-
-		lbf.registerSingleton("bd3", new Object());
-		assertThat(lbf.getBeanNamesForType(NestedTestBean.class)).isSameAs(nestedBeanNames);
-		assertThat(lbf.getBeanNamesForType(Object.class)).containsExactly(StringUtils.addStringToArray(allBeanNames, "bd3"));
-	}
-
 
 	private int registerBeanDefinitions(Properties p) {
 		return registerBeanDefinitions(p, null);
@@ -3446,7 +3418,7 @@ class DefaultListableBeanFactoryTests {
 		}
 
 		public TestBean create() {
-			DerivedTestBean tb = new DerivedTestBean();
+			TestBean tb = new TestBean();
 			tb.setName(this.name);
 			return tb;
 		}
@@ -3674,11 +3646,11 @@ class DefaultListableBeanFactoryTests {
 
 		private FactoryBean<?> factoryBean;
 
-		public FactoryBean<?> getFactoryBean() {
+		public final FactoryBean<?> getFactoryBean() {
 			return this.factoryBean;
 		}
 
-		public void setFactoryBean(FactoryBean<?> factoryBean) {
+		public final void setFactoryBean(final FactoryBean<?> factoryBean) {
 			this.factoryBean = factoryBean;
 		}
 	}

spring-beans/src/test/java/org/springframework/beans/factory/aot/InstanceSupplierCodeGeneratorTests.java

@@ -418,16 +418,6 @@ class InstanceSupplierCodeGeneratorTests {
 			compileAndCheckWarnings(beanDefinition);
 		}
 
-		@Test
-		void generateWhenTargetFactoryMethodIsProtectedAndReturnTypeIsDeprecated() {
-			BeanDefinition beanDefinition = BeanDefinitionBuilder
-					.rootBeanDefinition(DeprecatedBean.class)
-					.setFactoryMethodOnBean("deprecatedReturnTypeProtected", "config").getBeanDefinition();
-			beanFactory.registerBeanDefinition("config", BeanDefinitionBuilder
-					.genericBeanDefinition(DeprecatedMemberConfiguration.class).getBeanDefinition());
-			compileAndCheckWarnings(beanDefinition);
-		}
-
 		private void compileAndCheckWarnings(BeanDefinition beanDefinition) {
 			assertThatNoException().isThrownBy(() -> compile(TEST_COMPILER, beanDefinition,
 					((instanceSupplier, compiled) -> {})));
@@ -474,26 +464,6 @@ class InstanceSupplierCodeGeneratorTests {
 			compileAndCheckWarnings(beanDefinition);
 		}
 
-		@Test
-		void generateWhenTargetFactoryMethodReturnTypeIsDeprecatedForRemoval() {
-			BeanDefinition beanDefinition = BeanDefinitionBuilder
-					.rootBeanDefinition(DeprecatedForRemovalBean.class)
-					.setFactoryMethodOnBean("deprecatedReturnType", "config").getBeanDefinition();
-			beanFactory.registerBeanDefinition("config", BeanDefinitionBuilder
-					.genericBeanDefinition(DeprecatedForRemovalMemberConfiguration.class).getBeanDefinition());
-			compileAndCheckWarnings(beanDefinition);
-		}
-
-		@Test
-		void generateWhenTargetFactoryMethodIsProtectedAndReturnTypeIsDeprecatedForRemoval() {
-			BeanDefinition beanDefinition = BeanDefinitionBuilder
-					.rootBeanDefinition(DeprecatedForRemovalBean.class)
-					.setFactoryMethodOnBean("deprecatedReturnTypeProtected", "config").getBeanDefinition();
-			beanFactory.registerBeanDefinition("config", BeanDefinitionBuilder
-					.genericBeanDefinition(DeprecatedForRemovalMemberConfiguration.class).getBeanDefinition());
-			compileAndCheckWarnings(beanDefinition);
-		}
-
 		private void compileAndCheckWarnings(BeanDefinition beanDefinition) {
 			assertThatNoException().isThrownBy(() -> compile(TEST_COMPILER, beanDefinition,
 					((instanceSupplier, compiled) -> {})));

spring-beans/src/test/java/org/springframework/beans/factory/support/RootBeanDefinitionTests.java

@@ -17,7 +17,6 @@
 package org.springframework.beans.factory.support;
 
 import java.lang.reflect.Method;
-import java.util.concurrent.ExecutorService;
 
 import org.junit.jupiter.api.Test;
 
@@ -60,38 +59,13 @@ class RootBeanDefinitionTests {
 	}
 
 	@Test
-	void resolveDestroyMethodWithMatchingCandidateReplacedForCloseMethod() {
+	void resolveDestroyMethodWithMatchingCandidateReplacedInferredVaue() {
 		RootBeanDefinition beanDefinition = new RootBeanDefinition(BeanWithCloseMethod.class);
 		beanDefinition.setDestroyMethodName(AbstractBeanDefinition.INFER_METHOD);
 		beanDefinition.resolveDestroyMethodIfNecessary();
 		assertThat(beanDefinition.getDestroyMethodNames()).containsExactly("close");
 	}
 
-	@Test
-	void resolveDestroyMethodWithMatchingCandidateReplacedForShutdownMethod() {
-		RootBeanDefinition beanDefinition = new RootBeanDefinition(BeanWithShutdownMethod.class);
-		beanDefinition.setDestroyMethodName(AbstractBeanDefinition.INFER_METHOD);
-		beanDefinition.resolveDestroyMethodIfNecessary();
-		assertThat(beanDefinition.getDestroyMethodNames()).containsExactly("shutdown");
-	}
-
-	@Test
-	void resolveDestroyMethodWithMatchingCandidateReplacedForExecutorService() {
-		RootBeanDefinition beanDefinition = new RootBeanDefinition(BeanImplementingExecutorService.class);
-		beanDefinition.setDestroyMethodName(AbstractBeanDefinition.INFER_METHOD);
-		beanDefinition.resolveDestroyMethodIfNecessary();
-		assertThat(beanDefinition.getDestroyMethodNames()).containsExactly("shutdown");
-		// even on JDK 19+ where the ExecutorService interface declares a default AutoCloseable implementation
-	}
-
-	@Test
-	void resolveDestroyMethodWithMatchingCandidateReplacedForAutoCloseableExecutorService() {
-		RootBeanDefinition beanDefinition = new RootBeanDefinition(BeanImplementingExecutorServiceAndAutoCloseable.class);
-		beanDefinition.setDestroyMethodName(AbstractBeanDefinition.INFER_METHOD);
-		beanDefinition.resolveDestroyMethodIfNecessary();
-		assertThat(beanDefinition.getDestroyMethodNames()).containsExactly("close");
-	}
-
 	@Test
 	void resolveDestroyMethodWithNoCandidateSetDestroyMethodNameToNull() {
 		RootBeanDefinition beanDefinition = new RootBeanDefinition(BeanWithNoDestroyMethod.class);
@@ -116,25 +90,6 @@ class RootBeanDefinitionTests {
 	}
 
 
-	static class BeanWithShutdownMethod {
-
-		public void shutdown() {
-		}
-	}
-
-
-	abstract static class BeanImplementingExecutorService implements ExecutorService {
-	}
-
-
-	abstract static class BeanImplementingExecutorServiceAndAutoCloseable implements ExecutorService, AutoCloseable {
-
-		@Override
-		public void close() {
-		}
-	}
-
-
 	static class BeanWithNoDestroyMethod {
 	}
 

spring-beans/src/testFixtures/java/org/springframework/beans/testfixture/beans/factory/generator/deprecation/DeprecatedForRemovalMemberConfiguration.java

@@ -33,14 +33,4 @@ public class DeprecatedForRemovalMemberConfiguration {
 		return bean.toString();
 	}
 
-	@SuppressWarnings("removal")
-	public DeprecatedForRemovalBean deprecatedReturnType() {
-		return new DeprecatedForRemovalBean();
-	}
-
-	@SuppressWarnings("removal")
-	DeprecatedForRemovalBean deprecatedReturnTypeProtected() {
-		return new DeprecatedForRemovalBean();
-	}
-
 }

spring-beans/src/testFixtures/java/org/springframework/beans/testfixture/beans/factory/generator/deprecation/DeprecatedMemberConfiguration.java

@@ -38,9 +38,4 @@ public class DeprecatedMemberConfiguration {
 		return new DeprecatedBean();
 	}
 
-	@SuppressWarnings("deprecation")
-	DeprecatedBean deprecatedReturnTypeProtected() {
-		return new DeprecatedBean();
-	}
-
 }

spring-context-indexer/src/main/java/org/springframework/context/index/processor/CandidateComponentsIndexer.java

@@ -37,13 +37,12 @@ import javax.lang.model.element.TypeElement;
 
 /**
  * Annotation {@link Processor} that writes a {@link CandidateComponentsMetadata}
- * file for Spring components.
+ * file for spring components.
  *
  * @author Stephane Nicoll
  * @author Juergen Hoeller
  * @since 5.0
- * @deprecated as of 6.1, in favor of the AOT engine and the forthcoming
- * support for an AOT-generated Spring components index
+ * @deprecated as of 6.1, in favor of the AOT engine.
  */
 @Deprecated(since = "6.1", forRemoval = true)
 public class CandidateComponentsIndexer implements Processor {

spring-context-indexer/src/main/java/org/springframework/context/index/processor/StandardStereotypesProvider.java

@@ -25,8 +25,8 @@ import javax.lang.model.element.ElementKind;
 
 /**
  * A {@link StereotypesProvider} that extracts a stereotype for each
- * {@code jakarta.*} or {@code javax.*} annotation <i>present</i>
- * on a class or interface.
+ * {@code jakarta.*} or {@code javax.*} annotation <i>present</i> on a class or
+ * interface.
  *
  * @author Stephane Nicoll
  * @since 5.0

spring-context/src/main/java/org/springframework/context/MessageSource.java

@@ -55,7 +55,7 @@ public interface MessageSource {
 	 * @see java.text.MessageFormat
 	 */
 	@Nullable
-	String getMessage(String code, @Nullable Object[] args, @Nullable String defaultMessage, @Nullable Locale locale);
+	String getMessage(String code, @Nullable Object[] args, @Nullable String defaultMessage, Locale locale);
 
 	/**
 	 * Try to resolve the message. Treat as an error if the message can't be found.
@@ -71,7 +71,7 @@ public interface MessageSource {
 	 * @see #getMessage(MessageSourceResolvable, Locale)
 	 * @see java.text.MessageFormat
 	 */
-	String getMessage(String code, @Nullable Object[] args, @Nullable Locale locale) throws NoSuchMessageException;
+	String getMessage(String code, @Nullable Object[] args, Locale locale) throws NoSuchMessageException;
 
 	/**
 	 * Try to resolve the message using all the attributes contained within the
@@ -91,6 +91,6 @@ public interface MessageSource {
 	 * @see MessageSourceResolvable#getDefaultMessage()
 	 * @see java.text.MessageFormat
 	 */
-	String getMessage(MessageSourceResolvable resolvable, @Nullable Locale locale) throws NoSuchMessageException;
+	String getMessage(MessageSourceResolvable resolvable, Locale locale) throws NoSuchMessageException;
 
 }

spring-context/src/main/java/org/springframework/context/annotation/AdviceMode.java

@@ -17,18 +17,19 @@
 package org.springframework.context.annotation;
 
 /**
- * Enumeration used to determine whether JDK/CGLIB proxy-based or
+ * Enumeration used to determine whether JDK proxy-based or
  * AspectJ weaving-based advice should be applied.
  *
  * @author Chris Beams
  * @since 3.1
- * @see org.springframework.scheduling.annotation.AsyncConfigurationSelector#selectImports
  * @see org.springframework.scheduling.annotation.EnableAsync#mode()
+ * @see org.springframework.scheduling.annotation.AsyncConfigurationSelector#selectImports
+ * @see org.springframework.transaction.annotation.EnableTransactionManagement#mode()
  */
 public enum AdviceMode {
 
 	/**
-	 * JDK/CGLIB proxy-based advice.
+	 * JDK proxy-based advice.
 	 */
 	PROXY,
 

spring-context/src/main/java/org/springframework/context/annotation/ClassPathScanningCandidateComponentProvider.java

@@ -90,6 +90,7 @@ import org.springframework.util.ClassUtils;
  * @see ScannedGenericBeanDefinition
  * @see CandidateComponentsIndex
  */
+@SuppressWarnings("removal") // components index
 public class ClassPathScanningCandidateComponentProvider implements EnvironmentCapable, ResourceLoaderAware {
 
 	static final String DEFAULT_RESOURCE_PATTERN = "**/*.class";
@@ -451,9 +452,9 @@ public class ClassPathScanningCandidateComponentProvider implements EnvironmentC
 	private Set<BeanDefinition> scanCandidateComponents(String basePackage) {
 		Set<BeanDefinition> candidates = new LinkedHashSet<>();
 		try {
-			String packageSearchPattern = ResourcePatternResolver.CLASSPATH_ALL_URL_PREFIX +
+			String packageSearchPath = ResourcePatternResolver.CLASSPATH_ALL_URL_PREFIX +
 					resolveBasePackage(basePackage) + '/' + this.resourcePattern;
-			Resource[] resources = getResourcePatternResolver().getResources(packageSearchPattern);
+			Resource[] resources = getResourcePatternResolver().getResources(packageSearchPath);
 			boolean traceEnabled = logger.isTraceEnabled();
 			boolean debugEnabled = logger.isDebugEnabled();
 			for (Resource resource : resources) {
@@ -536,13 +537,13 @@ public class ClassPathScanningCandidateComponentProvider implements EnvironmentC
 	 * @return whether the class qualifies as a candidate component
 	 */
 	protected boolean isCandidateComponent(MetadataReader metadataReader) throws IOException {
-		for (TypeFilter filter : this.excludeFilters) {
-			if (filter.match(metadataReader, getMetadataReaderFactory())) {
+		for (TypeFilter tf : this.excludeFilters) {
+			if (tf.match(metadataReader, getMetadataReaderFactory())) {
 				return false;
 			}
 		}
-		for (TypeFilter filter : this.includeFilters) {
-			if (filter.match(metadataReader, getMetadataReaderFactory())) {
+		for (TypeFilter tf : this.includeFilters) {
+			if (tf.match(metadataReader, getMetadataReaderFactory())) {
 				return isConditionMatch(metadataReader);
 			}
 		}

spring-context/src/main/java/org/springframework/context/annotation/ConfigurationClassBeanDefinitionReader.java

@@ -157,17 +157,9 @@ class ConfigurationClassBeanDefinitionReader {
 
 		ScopeMetadata scopeMetadata = scopeMetadataResolver.resolveScopeMetadata(configBeanDef);
 		configBeanDef.setScope(scopeMetadata.getScopeName());
-
-		String configBeanName;
-		try {
-			configBeanName = this.importBeanNameGenerator.generateBeanName(configBeanDef, this.registry);
-		}
-		catch (IllegalArgumentException ex) {
-			throw new IllegalStateException("Failed to generate bean name for imported class '" +
-					configClass.getMetadata().getClassName() + "'", ex);
-		}
-
+		String configBeanName = this.importBeanNameGenerator.generateBeanName(configBeanDef, this.registry);
 		AnnotationConfigUtils.processCommonDefinitionAnnotations(configBeanDef, metadata);
+
 		BeanDefinitionHolder definitionHolder = new BeanDefinitionHolder(configBeanDef, configBeanName);
 		definitionHolder = AnnotationConfigUtils.applyScopedProxyMode(scopeMetadata, definitionHolder, this.registry);
 		this.registry.registerBeanDefinition(definitionHolder.getBeanName(), definitionHolder.getBeanDefinition());

spring-context/src/main/java/org/springframework/context/annotation/ReflectiveScan.java

@@ -51,9 +51,9 @@ import org.springframework.core.annotation.AliasFor;
  * ignored.
  *
  * @author Stephane Nicoll
- * @since 6.2
  * @see Reflective @Reflective
  * @see RegisterReflection @RegisterReflection
+ * @since 6.2
  */
 @Retention(RetentionPolicy.RUNTIME)
 @Target(ElementType.TYPE)

spring-context/src/main/java/org/springframework/context/aot/AotApplicationContextInitializer.java

@@ -63,13 +63,13 @@ public interface AotApplicationContextInitializer<C extends ConfigurableApplicat
 
 	private static <C extends ConfigurableApplicationContext> void initialize(
 			C applicationContext, String... initializerClassNames) {
-
 		Log logger = LogFactory.getLog(AotApplicationContextInitializer.class);
 		ClassLoader classLoader = applicationContext.getClassLoader();
 		logger.debug("Initializing ApplicationContext with AOT");
 		for (String initializerClassName : initializerClassNames) {
 			logger.trace(LogMessage.format("Applying %s", initializerClassName));
-			instantiateInitializer(initializerClassName, classLoader).initialize(applicationContext);
+			instantiateInitializer(initializerClassName, classLoader)
+					.initialize(applicationContext);
 		}
 	}
 

spring-context/src/main/java/org/springframework/context/event/EventListener.java

@@ -101,9 +101,10 @@ public @interface EventListener {
 
 	/**
 	 * The event classes that this listener handles.
-	 * <p>The annotated method may optionally accept a single parameter
-	 * of the given event class, or of a common base class or interface
-	 * for all given event classes.
+	 * <p>If this attribute is specified with a single value, the
+	 * annotated method may optionally accept a single parameter.
+	 * However, if this attribute is specified with multiple values,
+	 * the annotated method must <em>not</em> declare any parameters.
 	 */
 	@AliasFor("value")
 	Class<?>[] classes() default {};

spring-context/src/main/java/org/springframework/context/index/CandidateComponentsIndex.java

@@ -45,7 +45,9 @@ import org.springframework.util.MultiValueMap;
  *
  * @author Stephane Nicoll
  * @since 5.0
+ * @deprecated as of 6.1, in favor of the AOT engine.
  */
+@Deprecated(since = "6.1", forRemoval = true)
 public class CandidateComponentsIndex {
 
 	private static final AntPathMatcher pathMatcher = new AntPathMatcher(".");
@@ -81,7 +83,7 @@ public class CandidateComponentsIndex {
 	public Set<String> getCandidateTypes(String basePackage, String stereotype) {
 		List<Entry> candidates = this.index.get(stereotype);
 		if (candidates != null) {
-			return candidates.stream()
+			return candidates.parallelStream()
 					.filter(t -> t.match(basePackage))
 					.map(t -> t.type)
 					.collect(Collectors.toSet());
@@ -92,7 +94,7 @@ public class CandidateComponentsIndex {
 
 	private static class Entry {
 
-		final String type;
+		private final String type;
 
 		private final String packageName;
 

spring-context/src/main/java/org/springframework/context/index/CandidateComponentsIndexLoader.java

@@ -38,7 +38,10 @@ import org.springframework.util.ConcurrentReferenceHashMap;
  *
  * @author Stephane Nicoll
  * @since 5.0
+ * @deprecated as of 6.1, in favor of the AOT engine.
  */
+@Deprecated(since = "6.1", forRemoval = true)
+@SuppressWarnings("removal")
 public final class CandidateComponentsIndexLoader {
 
 	/**

spring-context/src/main/java/org/springframework/context/support/AbstractApplicationContext.java

@@ -936,9 +936,6 @@ public abstract class AbstractApplicationContext extends DefaultResourceLoader
 	 */
 	@SuppressWarnings("unchecked")
 	protected void finishBeanFactoryInitialization(ConfigurableListableBeanFactory beanFactory) {
-		// Mark current thread for singleton instantiation with applied bootstrap locking.
-		beanFactory.prepareSingletonBootstrap();
-
 		// Initialize bootstrap executor for this context.
 		if (beanFactory.containsBean(BOOTSTRAP_EXECUTOR_BEAN_NAME) &&
 				beanFactory.isTypeMatch(BOOTSTRAP_EXECUTOR_BEAN_NAME, Executor.class)) {
@@ -1505,17 +1502,17 @@ public abstract class AbstractApplicationContext extends DefaultResourceLoader
 
 	@Override
 	@Nullable
-	public String getMessage(String code, @Nullable Object[] args, @Nullable String defaultMessage, @Nullable Locale locale) {
+	public String getMessage(String code, @Nullable Object[] args, @Nullable String defaultMessage, Locale locale) {
 		return getMessageSource().getMessage(code, args, defaultMessage, locale);
 	}
 
 	@Override
-	public String getMessage(String code, @Nullable Object[] args, @Nullable Locale locale) throws NoSuchMessageException {
+	public String getMessage(String code, @Nullable Object[] args, Locale locale) throws NoSuchMessageException {
 		return getMessageSource().getMessage(code, args, locale);
 	}
 
 	@Override
-	public String getMessage(MessageSourceResolvable resolvable, @Nullable Locale locale) throws NoSuchMessageException {
+	public String getMessage(MessageSourceResolvable resolvable, Locale locale) throws NoSuchMessageException {
 		return getMessageSource().getMessage(resolvable, locale);
 	}
 

spring-context/src/main/java/org/springframework/context/support/AbstractMessageSource.java

@@ -138,7 +138,7 @@ public abstract class AbstractMessageSource extends MessageSourceSupport impleme
 
 	@Override
 	@Nullable
-	public final String getMessage(String code, @Nullable Object[] args, @Nullable String defaultMessage, @Nullable Locale locale) {
+	public final String getMessage(String code, @Nullable Object[] args, @Nullable String defaultMessage, Locale locale) {
 		String msg = getMessageInternal(code, args, locale);
 		if (msg != null) {
 			return msg;
@@ -150,7 +150,7 @@ public abstract class AbstractMessageSource extends MessageSourceSupport impleme
 	}
 
 	@Override
-	public final String getMessage(String code, @Nullable Object[] args, @Nullable Locale locale) throws NoSuchMessageException {
+	public final String getMessage(String code, @Nullable Object[] args, Locale locale) throws NoSuchMessageException {
 		String msg = getMessageInternal(code, args, locale);
 		if (msg != null) {
 			return msg;
@@ -159,16 +159,11 @@ public abstract class AbstractMessageSource extends MessageSourceSupport impleme
 		if (fallback != null) {
 			return fallback;
 		}
-		if (locale == null ) {
-			throw new NoSuchMessageException(code);
-		}
-		else {
-			throw new NoSuchMessageException(code, locale);
-		}
+		throw new NoSuchMessageException(code, locale);
 	}
 
 	@Override
-	public final String getMessage(MessageSourceResolvable resolvable, @Nullable Locale locale) throws NoSuchMessageException {
+	public final String getMessage(MessageSourceResolvable resolvable, Locale locale) throws NoSuchMessageException {
 		String[] codes = resolvable.getCodes();
 		if (codes != null) {
 			for (String code : codes) {
@@ -182,13 +177,7 @@ public abstract class AbstractMessageSource extends MessageSourceSupport impleme
 		if (defaultMessage != null) {
 			return defaultMessage;
 		}
-		String code = !ObjectUtils.isEmpty(codes) ? codes[codes.length - 1] : "";
-		if (locale == null ) {
-			throw new NoSuchMessageException(code);
-		}
-		else {
-			throw new NoSuchMessageException(code, locale);
-		}
+		throw new NoSuchMessageException(!ObjectUtils.isEmpty(codes) ? codes[codes.length - 1] : "", locale);
 	}
 
 
@@ -295,7 +284,7 @@ public abstract class AbstractMessageSource extends MessageSourceSupport impleme
 	 * @see #getDefaultMessage(String)
 	 */
 	@Nullable
-	protected String getDefaultMessage(MessageSourceResolvable resolvable, @Nullable Locale locale) {
+	protected String getDefaultMessage(MessageSourceResolvable resolvable, Locale locale) {
 		String defaultMessage = resolvable.getDefaultMessage();
 		String[] codes = resolvable.getCodes();
 		if (defaultMessage != null) {
@@ -342,7 +331,7 @@ public abstract class AbstractMessageSource extends MessageSourceSupport impleme
 	 * @return an array of arguments with any MessageSourceResolvables resolved
 	 */
 	@Override
-	protected Object[] resolveArguments(@Nullable Object[] args, @Nullable Locale locale) {
+	protected Object[] resolveArguments(@Nullable Object[] args, Locale locale) {
 		if (ObjectUtils.isEmpty(args)) {
 			return super.resolveArguments(args, locale);
 		}

spring-context/src/main/java/org/springframework/context/support/DelegatingMessageSource.java

@@ -55,7 +55,7 @@ public class DelegatingMessageSource extends MessageSourceSupport implements Hie
 
 	@Override
 	@Nullable
-	public String getMessage(String code, @Nullable Object[] args, @Nullable String defaultMessage, @Nullable Locale locale) {
+	public String getMessage(String code, @Nullable Object[] args, @Nullable String defaultMessage, Locale locale) {
 		if (this.parentMessageSource != null) {
 			return this.parentMessageSource.getMessage(code, args, defaultMessage, locale);
 		}
@@ -68,22 +68,17 @@ public class DelegatingMessageSource extends MessageSourceSupport implements Hie
 	}
 
 	@Override
-	public String getMessage(String code, @Nullable Object[] args, @Nullable Locale locale) throws NoSuchMessageException {
+	public String getMessage(String code, @Nullable Object[] args, Locale locale) throws NoSuchMessageException {
 		if (this.parentMessageSource != null) {
 			return this.parentMessageSource.getMessage(code, args, locale);
 		}
 		else {
-			if (locale == null) {
-				throw new NoSuchMessageException(code);
-			}
-			else {
-				throw new NoSuchMessageException(code, locale);
-			}
+			throw new NoSuchMessageException(code, locale);
 		}
 	}
 
 	@Override
-	public String getMessage(MessageSourceResolvable resolvable, @Nullable Locale locale) throws NoSuchMessageException {
+	public String getMessage(MessageSourceResolvable resolvable, Locale locale) throws NoSuchMessageException {
 		if (this.parentMessageSource != null) {
 			return this.parentMessageSource.getMessage(resolvable, locale);
 		}
@@ -93,12 +88,7 @@ public class DelegatingMessageSource extends MessageSourceSupport implements Hie
 			}
 			String[] codes = resolvable.getCodes();
 			String code = (codes != null && codes.length > 0 ? codes[0] : "");
-			if (locale == null) {
-				throw new NoSuchMessageException(code);
-			}
-			else {
-				throw new NoSuchMessageException(code, locale);
-			}
+			throw new NoSuchMessageException(code, locale);
 		}
 	}
 

spring-context/src/main/java/org/springframework/context/support/MessageSourceSupport.java

@@ -98,7 +98,7 @@ public abstract class MessageSourceSupport {
 	 * @return the rendered default message (with resolved arguments)
 	 * @see #formatMessage(String, Object[], java.util.Locale)
 	 */
-	protected String renderDefaultMessage(String defaultMessage, @Nullable Object[] args, @Nullable Locale locale) {
+	protected String renderDefaultMessage(String defaultMessage, @Nullable Object[] args, Locale locale) {
 		return formatMessage(defaultMessage, args, locale);
 	}
 
@@ -112,7 +112,7 @@ public abstract class MessageSourceSupport {
 	 * @param locale the Locale used for formatting
 	 * @return the formatted message (with resolved arguments)
 	 */
-	protected String formatMessage(String msg, @Nullable Object[] args, @Nullable Locale locale) {
+	protected String formatMessage(String msg, @Nullable Object[] args, Locale locale) {
 		if (!isAlwaysUseMessageFormat() && ObjectUtils.isEmpty(args)) {
 			return msg;
 		}
@@ -146,7 +146,7 @@ public abstract class MessageSourceSupport {
 	 * @param locale the Locale to create a {@code MessageFormat} for
 	 * @return the {@code MessageFormat} instance
 	 */
-	protected MessageFormat createMessageFormat(String msg, @Nullable Locale locale) {
+	protected MessageFormat createMessageFormat(String msg, Locale locale) {
 		return new MessageFormat(msg, locale);
 	}
 
@@ -158,7 +158,7 @@ public abstract class MessageSourceSupport {
 	 * @param locale the Locale to resolve against
 	 * @return the resolved argument array
 	 */
-	protected Object[] resolveArguments(@Nullable Object[] args, @Nullable Locale locale) {
+	protected Object[] resolveArguments(@Nullable Object[] args, Locale locale) {
 		return (args != null ? args : new Object[0]);
 	}
 

spring-context/src/main/java/org/springframework/scheduling/annotation/AsyncAnnotationBeanPostProcessor.java

@@ -87,6 +87,7 @@ public class AsyncAnnotationBeanPostProcessor extends AbstractBeanFactoryAwareAd
 	private Class<? extends Annotation> asyncAnnotationType;
 
 
+
 	public AsyncAnnotationBeanPostProcessor() {
 		setBeforeExistingAdvisors(true);
 	}

spring-context/src/main/java/org/springframework/scheduling/concurrent/SimpleAsyncTaskScheduler.java

@@ -376,7 +376,7 @@ public class SimpleAsyncTaskScheduler extends SimpleAsyncTaskExecutor implements
 
 	@Override
 	public boolean isRunning() {
-		return (this.triggerLifecycle.isRunning() || this.fixedDelayLifecycle.isRunning());
+		return this.triggerLifecycle.isRunning();
 	}
 
 	@Override

spring-context/src/main/java/org/springframework/scheduling/config/Task.java

@@ -50,9 +50,7 @@ public class Task {
 
 
 	/**
-	 * Return a {@link Runnable} that executes the underlying task.
-	 * <p>Note, this does not necessarily return the {@link Task#Task(Runnable) original runnable}
-	 * as it can be wrapped by the Framework for additional support.
+	 * Return the underlying task.
 	 */
 	public Runnable getRunnable() {
 		return this.runnable;

spring-context/src/main/java/org/springframework/validation/beanvalidation/MethodValidationInterceptor.java

@@ -69,6 +69,8 @@ import org.springframework.validation.method.ParameterValidationResult;
  * at the type level of the containing target class, applying to all public service methods
  * of that class. By default, JSR-303 will validate against its default group only.
  *
+ * <p>This functionality requires a Bean Validation 1.1+ provider.
+ *
  * @author Juergen Hoeller
  * @author Rossen Stoyanchev
  * @since 3.1

spring-context/src/main/java/org/springframework/validation/beanvalidation/MethodValidationPostProcessor.java

@@ -60,6 +60,8 @@ import org.springframework.validation.method.MethodValidationResult;
  * inline constraint annotations. Validation groups can be specified through {@code @Validated}
  * as well. By default, JSR-303 will validate against its default group only.
  *
+ * <p>This functionality requires a Bean Validation 1.1+ provider.
+ *
  * @author Juergen Hoeller
  * @since 3.1
  * @see MethodValidationInterceptor

spring-context/src/main/java/org/springframework/validation/beanvalidation/SpringConstraintValidatorFactory.java

@@ -56,6 +56,7 @@ public class SpringConstraintValidatorFactory implements ConstraintValidatorFact
 		return this.beanFactory.createBean(key);
 	}
 
+	// Bean Validation 1.1 releaseInstance method
 	@Override
 	public void releaseInstance(ConstraintValidator<?, ?> instance) {
 		this.beanFactory.destroyBean(instance);

spring-context/src/main/java/org/springframework/validation/beanvalidation/SpringValidatorAdapter.java

@@ -55,6 +55,8 @@ import org.springframework.validation.SmartValidator;
  * {@link CustomValidatorBean} and {@link LocalValidatorFactoryBean},
  * and as the primary implementation of the {@link SmartValidator} interface.
  *
+ * <p>This adapter is fully compatible with Bean Validation 1.1 as well as 2.0.
+ *
  * @author Juergen Hoeller
  * @author Sam Brannen
  * @since 3.0

spring-context/src/test/java/org/springframework/cache/interceptor/CacheOperationExpressionEvaluatorTests.java

@@ -60,6 +60,12 @@ class CacheOperationExpressionEvaluatorTests {
 	private final AnnotationCacheOperationSource source = new AnnotationCacheOperationSource();
 
 
+	private Collection<CacheOperation> getOps(String name) {
+		Method method = ReflectionUtils.findMethod(AnnotatedClass.class, name, Object.class, Object.class);
+		return this.source.getCacheOperations(method, AnnotatedClass.class);
+	}
+
+
 	@Test
 	void testMultipleCachingSource() {
 		Collection<CacheOperation> ops = getOps("multipleCaching");
@@ -138,12 +144,6 @@ class CacheOperationExpressionEvaluatorTests {
 		assertThat(value).isEqualTo(String.class.getName());
 	}
 
-
-	private Collection<CacheOperation> getOps(String name) {
-		Method method = ReflectionUtils.findMethod(AnnotatedClass.class, name, Object.class, Object.class);
-		return this.source.getCacheOperations(method, AnnotatedClass.class);
-	}
-
 	private EvaluationContext createEvaluationContext(Object result) {
 		return createEvaluationContext(result, null);
 	}

spring-context/src/test/java/org/springframework/cache/interceptor/CachePutEvaluationTests.java

@@ -104,7 +104,6 @@ class CachePutEvaluationTests {
 		assertThat(this.cache.get(anotherValue + 100).get()).as("Wrong value for @CachePut key").isEqualTo(anotherValue);
 	}
 
-
 	@Configuration
 	@EnableCaching
 	static class Config implements CachingConfigurer {
@@ -122,10 +121,8 @@ class CachePutEvaluationTests {
 
 	}
 
-
 	@CacheConfig("test")
 	public static class SimpleService {
-
 		private AtomicLong counter = new AtomicLong();
 
 		/**
@@ -147,5 +144,4 @@ class CachePutEvaluationTests {
 			return this.counter.getAndIncrement();
 		}
 	}
-
 }

spring-context/src/test/java/org/springframework/context/annotation/BackgroundBootstrapTests.java

@@ -31,7 +31,6 @@ import org.springframework.beans.factory.config.ConfigurableListableBeanFactory;
 import org.springframework.beans.factory.support.DefaultListableBeanFactory;
 import org.springframework.beans.testfixture.beans.TestBean;
 import org.springframework.context.ConfigurableApplicationContext;
-import org.springframework.context.weaving.LoadTimeWeaverAware;
 import org.springframework.core.SpringProperties;
 import org.springframework.core.testfixture.EnabledForTestGroups;
 import org.springframework.scheduling.concurrent.ThreadPoolTaskExecutor;
@@ -69,16 +68,6 @@ class BackgroundBootstrapTests {
 		ctx.close();
 	}
 
-	@Test
-	@Timeout(10)
-	@EnabledForTestGroups(LONG_RUNNING)
-	void bootstrapWithLoadTimeWeaverAware() {
-		ConfigurableApplicationContext ctx = new AnnotationConfigApplicationContext(LoadTimeWeaverAwareBeanConfig.class);
-		ctx.getBean("testBean1", TestBean.class);
-		ctx.getBean("testBean2", TestBean.class);
-		ctx.close();
-	}
-
 	@Test
 	@Timeout(10)
 	@EnabledForTestGroups(LONG_RUNNING)
@@ -277,50 +266,6 @@ class BackgroundBootstrapTests {
 	}
 
 
-	@Configuration(proxyBeanMethods = false)
-	static class LoadTimeWeaverAwareBeanConfig {
-
-		@Bean
-		LoadTimeWeaverAware loadTimeWeaverAware(ObjectProvider<TestBean> testBean1) {
-			Thread thread = new Thread(testBean1::getObject);
-			thread.start();
-			try {
-				thread.join();
-			}
-			catch (InterruptedException ex) {
-				Thread.currentThread().interrupt();
-			}
-			return (loadTimeWeaver -> {});
-		}
-
-		@Bean
-		public TestBean testBean1(TestBean testBean2) {
-			return new TestBean(testBean2);
-		}
-
-		@Bean @Lazy
-		public FactoryBean<TestBean> testBean2() {
-			try {
-				Thread.sleep(2000);
-			}
-			catch (InterruptedException ex) {
-				Thread.currentThread().interrupt();
-			}
-			TestBean testBean = new TestBean();
-			return new FactoryBean<>() {
-				@Override
-				public TestBean getObject() {
-					return testBean;
-				}
-				@Override
-				public Class<?> getObjectType() {
-					return testBean.getClass();
-				}
-			};
-		}
-	}
-
-
 	@Configuration(proxyBeanMethods = false)
 	static class StrictLockingBeanConfig {
 

spring-context/src/test/java/org/springframework/context/annotation/ClassPathBeanDefinitionScannerTests.java

@@ -61,7 +61,6 @@ class ClassPathBeanDefinitionScannerTests {
 		GenericApplicationContext context = new GenericApplicationContext();
 		ClassPathBeanDefinitionScanner scanner = new ClassPathBeanDefinitionScanner(context);
 		int beanCount = scanner.scan(BASE_PACKAGE);
-
 		assertThat(beanCount).isGreaterThanOrEqualTo(12);
 		assertThat(context.containsBean("serviceInvocationCounter")).isTrue();
 		assertThat(context.containsBean("fooServiceImpl")).isTrue();
@@ -74,8 +73,8 @@ class ClassPathBeanDefinitionScannerTests {
 		assertThat(context.containsBean(AnnotationConfigUtils.COMMON_ANNOTATION_PROCESSOR_BEAN_NAME)).isTrue();
 		assertThat(context.containsBean(AnnotationConfigUtils.EVENT_LISTENER_PROCESSOR_BEAN_NAME)).isTrue();
 		assertThat(context.containsBean(AnnotationConfigUtils.EVENT_LISTENER_FACTORY_BEAN_NAME)).isTrue();
-
 		context.refresh();
+
 		FooServiceImpl fooService = context.getBean("fooServiceImpl", FooServiceImpl.class);
 		assertThat(context.getDefaultListableBeanFactory().containsSingleton("myNamedComponent")).isTrue();
 		assertThat(fooService.foo(123)).isEqualTo("bar");
@@ -158,7 +157,6 @@ class ClassPathBeanDefinitionScannerTests {
 
 		ClassPathBeanDefinitionScanner scanner = new ClassPathBeanDefinitionScanner(context);
 		int beanCount = scanner.scan(BASE_PACKAGE);
-
 		assertThat(beanCount).isGreaterThanOrEqualTo(12);
 
 		ClassPathBeanDefinitionScanner scanner2 = new ClassPathBeanDefinitionScanner(context) {
@@ -184,8 +182,8 @@ class ClassPathBeanDefinitionScannerTests {
 		ClassPathBeanDefinitionScanner scanner = new ClassPathBeanDefinitionScanner(context);
 		scanner.setIncludeAnnotationConfig(false);
 		int beanCount = scanner.scan(BASE_PACKAGE);
-
 		assertThat(beanCount).isGreaterThanOrEqualTo(7);
+
 		assertThat(context.containsBean("serviceInvocationCounter")).isTrue();
 		assertThat(context.containsBean("fooServiceImpl")).isTrue();
 		assertThat(context.containsBean("stubFooDao")).isTrue();
@@ -484,14 +482,12 @@ class ClassPathBeanDefinitionScannerTests {
 		ClassPathBeanDefinitionScanner scanner = new ClassPathBeanDefinitionScanner(context);
 		scanner.setBeanNameGenerator(new TestBeanNameGenerator());
 		int beanCount = scanner.scan(BASE_PACKAGE);
-
 		assertThat(beanCount).isGreaterThanOrEqualTo(12);
-
 		context.refresh();
+
 		FooServiceImpl fooService = context.getBean("fooService", FooServiceImpl.class);
 		StaticListableBeanFactory myBf = (StaticListableBeanFactory) context.getBean("myBf");
 		MessageSource ms = (MessageSource) context.getBean("messageSource");
-
 		assertThat(fooService.isInitCalled()).isTrue();
 		assertThat(fooService.foo(123)).isEqualTo("bar");
 		assertThat(fooService.lookupFoo(123)).isEqualTo("bar");
@@ -513,10 +509,9 @@ class ClassPathBeanDefinitionScannerTests {
 		scanner.setIncludeAnnotationConfig(false);
 		scanner.setBeanNameGenerator(new TestBeanNameGenerator());
 		int beanCount = scanner.scan(BASE_PACKAGE);
-
 		assertThat(beanCount).isGreaterThanOrEqualTo(7);
-
 		context.refresh();
+
 		try {
 			context.getBean("fooService");
 		}
@@ -550,10 +545,9 @@ class ClassPathBeanDefinitionScannerTests {
 		scanner.setAutowireCandidatePatterns("*NoSuchDao");
 		scanner.scan(BASE_PACKAGE);
 		context.refresh();
-		assertThatExceptionOfType(BeanCreationException.class)
-				.isThrownBy(() -> context.getBean("fooService"))
-				.satisfies(ex ->
-						assertThat(ex.getMostSpecificCause()).isInstanceOf(NoSuchBeanDefinitionException.class));
+		assertThatExceptionOfType(BeanCreationException.class).isThrownBy(() ->
+				context.getBean("fooService"))
+			.satisfies(ex -> assertThat(ex.getMostSpecificCause()).isInstanceOf(NoSuchBeanDefinitionException.class));
 	}
 
 

spring-context/src/test/java/org/springframework/context/annotation/ConfigurationClassPostProcessorTests.java

@@ -409,14 +409,11 @@ class ConfigurationClassPostProcessorTests {
 		beanFactory.registerBeanDefinition("config", new RootBeanDefinition(SingletonBeanConfig.class));
 		beanFactory.setAllowBeanDefinitionOverriding(false);
 		ConfigurationClassPostProcessor pp = new ConfigurationClassPostProcessor();
-
 		assertThatExceptionOfType(BeanDefinitionStoreException.class)
 				.isThrownBy(() -> pp.postProcessBeanFactory(beanFactory))
-				.withMessageContainingAll(
-					"bar",
-					"SingletonBeanConfig",
-					TestBean.class.getName()
-				);
+				.withMessageContaining("bar")
+				.withMessageContaining("SingletonBeanConfig")
+				.withMessageContaining(TestBean.class.getName());
 	}
 
 	@Test  // gh-25430
@@ -425,13 +422,10 @@ class ConfigurationClassPostProcessorTests {
 		DefaultListableBeanFactory beanFactory = context.getDefaultListableBeanFactory();
 		beanFactory.setAllowBeanDefinitionOverriding(false);
 		context.register(FirstConfiguration.class, SecondConfiguration.class);
-
 		assertThatIllegalStateException().isThrownBy(context::refresh)
-				.withMessageContainingAll(
-					"alias 'taskExecutor'",
-					"name 'applicationTaskExecutor'",
-					"bean definition 'taskExecutor'"
-				);
+				.withMessageContaining("alias 'taskExecutor'")
+				.withMessageContaining("name 'applicationTaskExecutor'")
+				.withMessageContaining("bean definition 'taskExecutor'");
 		context.close();
 	}
 
@@ -984,7 +978,7 @@ class ConfigurationClassPostProcessorTests {
 	}
 
 	@Test
-	void selfReferenceExclusionForFactoryMethodOnSameBean() {
+	void testSelfReferenceExclusionForFactoryMethodOnSameBean() {
 		AutowiredAnnotationBeanPostProcessor bpp = new AutowiredAnnotationBeanPostProcessor();
 		bpp.setBeanFactory(beanFactory);
 		beanFactory.addBeanPostProcessor(bpp);
@@ -998,7 +992,7 @@ class ConfigurationClassPostProcessorTests {
 	}
 
 	@Test
-	void configWithDefaultMethods() {
+	void testConfigWithDefaultMethods() {
 		AutowiredAnnotationBeanPostProcessor bpp = new AutowiredAnnotationBeanPostProcessor();
 		bpp.setBeanFactory(beanFactory);
 		beanFactory.addBeanPostProcessor(bpp);
@@ -1012,7 +1006,7 @@ class ConfigurationClassPostProcessorTests {
 	}
 
 	@Test
-	void configWithDefaultMethodsUsingAsm() {
+	void testConfigWithDefaultMethodsUsingAsm() {
 		AutowiredAnnotationBeanPostProcessor bpp = new AutowiredAnnotationBeanPostProcessor();
 		bpp.setBeanFactory(beanFactory);
 		beanFactory.addBeanPostProcessor(bpp);
@@ -1026,7 +1020,7 @@ class ConfigurationClassPostProcessorTests {
 	}
 
 	@Test
-	void configWithFailingInit() {  // gh-23343
+	void testConfigWithFailingInit() {  // gh-23343
 		AutowiredAnnotationBeanPostProcessor bpp = new AutowiredAnnotationBeanPostProcessor();
 		bpp.setBeanFactory(beanFactory);
 		beanFactory.addBeanPostProcessor(bpp);
@@ -1040,7 +1034,7 @@ class ConfigurationClassPostProcessorTests {
 	}
 
 	@Test
-	void circularDependency() {
+	void testCircularDependency() {
 		AutowiredAnnotationBeanPostProcessor bpp = new AutowiredAnnotationBeanPostProcessor();
 		bpp.setBeanFactory(beanFactory);
 		beanFactory.addBeanPostProcessor(bpp);
@@ -1054,42 +1048,42 @@ class ConfigurationClassPostProcessorTests {
 	}
 
 	@Test
-	void circularDependencyWithApplicationContext() {
+	void testCircularDependencyWithApplicationContext() {
 		assertThatExceptionOfType(BeanCreationException.class)
 				.isThrownBy(() -> new AnnotationConfigApplicationContext(A.class, AStrich.class))
 				.withMessageContaining("Circular reference");
 	}
 
 	@Test
-	void prototypeArgumentThroughBeanMethodCall() {
+	void testPrototypeArgumentThroughBeanMethodCall() {
 		ConfigurableApplicationContext ctx = new AnnotationConfigApplicationContext(BeanArgumentConfigWithPrototype.class);
 		ctx.getBean(FooFactory.class).createFoo(new BarArgument());
 		ctx.close();
 	}
 
 	@Test
-	void singletonArgumentThroughBeanMethodCall() {
+	void testSingletonArgumentThroughBeanMethodCall() {
 		ConfigurableApplicationContext ctx = new AnnotationConfigApplicationContext(BeanArgumentConfigWithSingleton.class);
 		ctx.getBean(FooFactory.class).createFoo(new BarArgument());
 		ctx.close();
 	}
 
 	@Test
-	void nullArgumentThroughBeanMethodCall() {
+	void testNullArgumentThroughBeanMethodCall() {
 		ConfigurableApplicationContext ctx = new AnnotationConfigApplicationContext(BeanArgumentConfigWithNull.class);
 		ctx.getBean("aFoo");
 		ctx.close();
 	}
 
 	@Test
-	void injectionPointMatchForNarrowTargetReturnType() {
+	void testInjectionPointMatchForNarrowTargetReturnType() {
 		ConfigurableApplicationContext ctx = new AnnotationConfigApplicationContext(FooBarConfiguration.class);
 		assertThat(ctx.getBean(FooImpl.class).bar).isSameAs(ctx.getBean(BarImpl.class));
 		ctx.close();
 	}
 
 	@Test
-	void varargOnBeanMethod() {
+	void testVarargOnBeanMethod() {
 		ConfigurableApplicationContext ctx = new AnnotationConfigApplicationContext(VarargConfiguration.class, TestBean.class);
 		VarargConfiguration bean = ctx.getBean(VarargConfiguration.class);
 		assertThat(bean.testBeans).isNotNull();
@@ -1099,7 +1093,7 @@ class ConfigurationClassPostProcessorTests {
 	}
 
 	@Test
-	void emptyVarargOnBeanMethod() {
+	void testEmptyVarargOnBeanMethod() {
 		ConfigurableApplicationContext ctx = new AnnotationConfigApplicationContext(VarargConfiguration.class);
 		VarargConfiguration bean = ctx.getBean(VarargConfiguration.class);
 		assertThat(bean.testBeans).isNotNull();
@@ -1108,7 +1102,7 @@ class ConfigurationClassPostProcessorTests {
 	}
 
 	@Test
-	void collectionArgumentOnBeanMethod() {
+	void testCollectionArgumentOnBeanMethod() {
 		ConfigurableApplicationContext ctx = new AnnotationConfigApplicationContext(CollectionArgumentConfiguration.class, TestBean.class);
 		CollectionArgumentConfiguration bean = ctx.getBean(CollectionArgumentConfiguration.class);
 		assertThat(bean.testBeans).containsExactly(ctx.getBean(TestBean.class));
@@ -1116,7 +1110,7 @@ class ConfigurationClassPostProcessorTests {
 	}
 
 	@Test
-	void emptyCollectionArgumentOnBeanMethod() {
+	void testEmptyCollectionArgumentOnBeanMethod() {
 		ConfigurableApplicationContext ctx = new AnnotationConfigApplicationContext(CollectionArgumentConfiguration.class);
 		CollectionArgumentConfiguration bean = ctx.getBean(CollectionArgumentConfiguration.class);
 		assertThat(bean.testBeans).isEmpty();
@@ -1124,7 +1118,7 @@ class ConfigurationClassPostProcessorTests {
 	}
 
 	@Test
-	void mapArgumentOnBeanMethod() {
+	void testMapArgumentOnBeanMethod() {
 		ConfigurableApplicationContext ctx = new AnnotationConfigApplicationContext(MapArgumentConfiguration.class, DummyRunnable.class);
 		MapArgumentConfiguration bean = ctx.getBean(MapArgumentConfiguration.class);
 		assertThat(bean.testBeans).hasSize(1).containsValue(ctx.getBean(Runnable.class));
@@ -1132,7 +1126,7 @@ class ConfigurationClassPostProcessorTests {
 	}
 
 	@Test
-	void emptyMapArgumentOnBeanMethod() {
+	void testEmptyMapArgumentOnBeanMethod() {
 		ConfigurableApplicationContext ctx = new AnnotationConfigApplicationContext(MapArgumentConfiguration.class);
 		MapArgumentConfiguration bean = ctx.getBean(MapArgumentConfiguration.class);
 		assertThat(bean.testBeans).isEmpty();
@@ -1140,7 +1134,7 @@ class ConfigurationClassPostProcessorTests {
 	}
 
 	@Test
-	void collectionInjectionFromSameConfigurationClass() {
+	void testCollectionInjectionFromSameConfigurationClass() {
 		ConfigurableApplicationContext ctx = new AnnotationConfigApplicationContext(CollectionInjectionConfiguration.class);
 		CollectionInjectionConfiguration bean = ctx.getBean(CollectionInjectionConfiguration.class);
 		assertThat(bean.testBeans).containsExactly(ctx.getBean(TestBean.class));
@@ -1148,7 +1142,7 @@ class ConfigurationClassPostProcessorTests {
 	}
 
 	@Test
-	void mapInjectionFromSameConfigurationClass() {
+	void testMapInjectionFromSameConfigurationClass() {
 		ConfigurableApplicationContext ctx = new AnnotationConfigApplicationContext(MapInjectionConfiguration.class);
 		MapInjectionConfiguration bean = ctx.getBean(MapInjectionConfiguration.class);
 		assertThat(bean.testBeans).containsOnly(Map.entry("testBean", ctx.getBean(Runnable.class)));
@@ -1156,20 +1150,20 @@ class ConfigurationClassPostProcessorTests {
 	}
 
 	@Test
-	void beanLookupFromSameConfigurationClass() {
+	void testBeanLookupFromSameConfigurationClass() {
 		ConfigurableApplicationContext ctx = new AnnotationConfigApplicationContext(BeanLookupConfiguration.class);
 		assertThat(ctx.getBean(BeanLookupConfiguration.class).getTestBean()).isSameAs(ctx.getBean(TestBean.class));
 		ctx.close();
 	}
 
 	@Test
-	void nameClashBetweenConfigurationClassAndBean() {
+	void testNameClashBetweenConfigurationClassAndBean() {
 		assertThatExceptionOfType(BeanDefinitionStoreException.class)
-				.isThrownBy(() -> new AnnotationConfigApplicationContext(MyTestBean.class));
+				.isThrownBy(() -> new AnnotationConfigApplicationContext(MyTestBean.class).getBean("myTestBean", TestBean.class));
 	}
 
 	@Test
-	void beanDefinitionRegistryPostProcessorConfig() {
+	void testBeanDefinitionRegistryPostProcessorConfig() {
 		ConfigurableApplicationContext ctx = new AnnotationConfigApplicationContext(BeanDefinitionRegistryPostProcessorConfig.class);
 		assertThat(ctx.getBean("myTestBean")).isInstanceOf(TestBean.class);
 		ctx.close();

spring-context/src/test/java/org/springframework/context/event/AnnotationDrivenEventListenerTests.java

@@ -16,7 +16,6 @@
 
 package org.springframework.context.event;
 
-import java.io.Serializable;
 import java.lang.annotation.ElementType;
 import java.lang.annotation.Retention;
 import java.lang.annotation.RetentionPolicy;
@@ -106,9 +105,8 @@ class AnnotationDrivenEventListenerTests {
 		this.eventCollector.assertTotalEventsCount(1);
 
 		this.eventCollector.clear();
-		TestEvent otherEvent = new TestEvent(this, Integer.valueOf(1));
-		this.context.publishEvent(otherEvent);
-		this.eventCollector.assertEvent(listener, otherEvent);
+		this.context.publishEvent(event);
+		this.eventCollector.assertEvent(listener, event);
 		this.eventCollector.assertTotalEventsCount(1);
 
 		context.getBean(ApplicationEventMulticaster.class).removeApplicationListeners(l ->
@@ -744,11 +742,6 @@ class AnnotationDrivenEventListenerTests {
 		public void handleString(String content) {
 			collectEvent(content);
 		}
-
-		@EventListener({Boolean.class, Integer.class})
-		public void handleBooleanOrInteger(Serializable content) {
-			collectEvent(content);
-		}
 	}
 
 
@@ -1016,8 +1009,6 @@ class AnnotationDrivenEventListenerTests {
 
 		void handleString(String payload);
 
-		void handleBooleanOrInteger(Serializable content);
-
 		void handleTimestamp(Long timestamp);
 
 		void handleRatio(Double ratio);
@@ -1040,12 +1031,6 @@ class AnnotationDrivenEventListenerTests {
 			super.handleString(payload);
 		}
 
-		@EventListener({Boolean.class, Integer.class})
-		@Override
-		public void handleBooleanOrInteger(Serializable content) {
-			super.handleBooleanOrInteger(content);
-		}
-
 		@ConditionalEvent("#root.event.timestamp > #p0")
 		@Override
 		public void handleTimestamp(Long timestamp) {

spring-context/src/test/java/org/springframework/context/event/test/TestEvent.java

@@ -18,12 +18,11 @@ package org.springframework.context.event.test;
 
 /**
  * @author Stephane Nicoll
- * @author Juergen Hoeller
  */
 @SuppressWarnings("serial")
 public class TestEvent extends IdentifiableApplicationEvent {
 
-	public final Object msg;
+	public final String msg;
 
 	public TestEvent(Object source, String id, String msg) {
 		super(source, id);
@@ -35,11 +34,6 @@ public class TestEvent extends IdentifiableApplicationEvent {
 		this.msg = msg;
 	}
 
-	public TestEvent(Object source, Integer msg) {
-		super(source);
-		this.msg = msg;
-	}
-
 	public TestEvent(Object source) {
 		this(source, "test");
 	}

spring-context/src/test/java/org/springframework/context/index/CandidateComponentsIndexLoaderTests.java

@@ -32,6 +32,8 @@ import static org.assertj.core.api.Assertions.assertThatIllegalStateException;
  *
  * @author Stephane Nicoll
  */
+@Deprecated
+@SuppressWarnings("removal")
 public class CandidateComponentsIndexLoaderTests {
 
 	@Test

spring-context/src/test/java/org/springframework/context/index/CandidateComponentsIndexTests.java

@@ -30,6 +30,8 @@ import static org.assertj.core.api.Assertions.assertThat;
  *
  * @author Stephane Nicoll
  */
+@Deprecated
+@SuppressWarnings("removal")
 public class CandidateComponentsIndexTests {
 
 	@Test

spring-core/src/main/java/org/springframework/cglib/proxy/Enhancer.java

@@ -1278,32 +1278,29 @@ public class Enhancer extends AbstractClassGenerator {
 				Signature bridgeTarget = (Signature) bridgeToTarget.get(method.getSignature());
 				if (bridgeTarget != null) {
 					// checkcast each argument against the target's argument types
-					Type[] argTypes = method.getSignature().getArgumentTypes();
-					Type[] targetTypes = bridgeTarget.getArgumentTypes();
-					for (int i = 0; i < targetTypes.length; i++) {
+					for (int i = 0; i < bridgeTarget.getArgumentTypes().length; i++) {
 						e.load_arg(i);
-						Type argType = argTypes[i];
-						Type target = targetTypes[i];
-						if (!target.equals(argType)) {
-							if (!TypeUtils.isPrimitive(target)) {
-								e.box(argType);
-							}
+						Type target = bridgeTarget.getArgumentTypes()[i];
+						if (!target.equals(method.getSignature().getArgumentTypes()[i])) {
 							e.checkcast(target);
 						}
 					}
 
 					e.invoke_virtual_this(bridgeTarget);
 
-					// Not necessary to cast if the target & bridge have the same return type.
 					Type retType = method.getSignature().getReturnType();
-					Type target = bridgeTarget.getReturnType();
-					if (!target.equals(retType)) {
-						if (!TypeUtils.isPrimitive(target)) {
-							e.unbox(retType);
-						}
-						else {
-							e.checkcast(retType);
-						}
+					// Not necessary to cast if the target & bridge have
+					// the same return type.
+					// (This conveniently includes void and primitive types,
+					// which would fail if casted.  It's not possible to
+					// covariant from boxed to unbox (or vice versa), so no having
+					// to box/unbox for bridges).
+					// TODO: It also isn't necessary to checkcast if the return is
+					// assignable from the target.  (This would happen if a subclass
+					// used covariant returns to narrow the return type within a bridge
+					// method.)
+					if (!retType.equals(bridgeTarget.getReturnType())) {
+						e.checkcast(retType);
 					}
 				}
 				else {

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

@@ -18,7 +18,6 @@ package org.springframework.core.annotation;
 
 import java.lang.annotation.Annotation;
 import java.lang.reflect.Method;
-import java.lang.reflect.Modifier;
 import java.util.ArrayList;
 import java.util.Arrays;
 import java.util.List;
@@ -39,7 +38,6 @@ import org.springframework.util.StringUtils;
  * interface-declared parameter annotations from the concrete target method.
  *
  * @author Juergen Hoeller
- * @author Sam Brannen
  * @since 6.1
  * @see #getMethodAnnotation(Class)
  * @see #getMethodParameters()
@@ -183,7 +181,7 @@ public class AnnotatedMethod {
 					clazz = null;
 				}
 				if (clazz != null) {
-					for (Method candidate : clazz.getDeclaredMethods()) {
+					for (Method candidate : clazz.getMethods()) {
 						if (isOverrideFor(candidate)) {
 							parameterAnnotations.add(candidate.getParameterAnnotations());
 						}
@@ -196,9 +194,8 @@ public class AnnotatedMethod {
 	}
 
 	private boolean isOverrideFor(Method candidate) {
-		if (Modifier.isPrivate(candidate.getModifiers()) ||
-				!candidate.getName().equals(this.method.getName()) ||
-				(candidate.getParameterCount() != this.method.getParameterCount())) {
+		if (!candidate.getName().equals(this.method.getName()) ||
+				candidate.getParameterCount() != this.method.getParameterCount()) {
 			return false;
 		}
 		Class<?>[] paramTypes = this.method.getParameterTypes();
@@ -207,7 +204,7 @@ public class AnnotatedMethod {
 		}
 		for (int i = 0; i < paramTypes.length; i++) {
 			if (paramTypes[i] !=
-					ResolvableType.forMethodParameter(candidate, i, this.method.getDeclaringClass()).toClass()) {
+					ResolvableType.forMethodParameter(candidate, i, this.method.getDeclaringClass()).resolve()) {
 				return false;
 			}
 		}

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

@@ -372,14 +372,14 @@ abstract class AnnotationsScanner {
 	private static boolean hasSameGenericTypeParameters(
 			Method rootMethod, Method candidateMethod, Class<?>[] rootParameterTypes) {
 
-		Class<?> rootDeclaringClass = rootMethod.getDeclaringClass();
+		Class<?> sourceDeclaringClass = rootMethod.getDeclaringClass();
 		Class<?> candidateDeclaringClass = candidateMethod.getDeclaringClass();
-		if (!candidateDeclaringClass.isAssignableFrom(rootDeclaringClass)) {
+		if (!candidateDeclaringClass.isAssignableFrom(sourceDeclaringClass)) {
 			return false;
 		}
 		for (int i = 0; i < rootParameterTypes.length; i++) {
 			Class<?> resolvedParameterType = ResolvableType.forMethodParameter(
-					candidateMethod, i, rootDeclaringClass).toClass();
+					candidateMethod, i, sourceDeclaringClass).resolve();
 			if (rootParameterTypes[i] != resolvedParameterType) {
 				return false;
 			}

spring-core/src/main/java/org/springframework/core/codec/AbstractCharSequenceDecoder.java

@@ -175,18 +175,14 @@ public abstract class AbstractCharSequenceDecoder<T extends CharSequence> extend
 	public final T decode(DataBuffer dataBuffer, ResolvableType elementType,
 			@Nullable MimeType mimeType, @Nullable Map<String, Object> hints) {
 
-		try {
-			Charset charset = getCharset(mimeType);
-			T value = decodeInternal(dataBuffer, charset);
-			LogFormatUtils.traceDebug(logger, traceOn -> {
-				String formatted = LogFormatUtils.formatValue(value, !traceOn);
-				return Hints.getLogPrefix(hints) + "Decoded " + formatted;
-			});
-			return value;
-		}
-		finally {
-			DataBufferUtils.release(dataBuffer);
-		}
+		Charset charset = getCharset(mimeType);
+		T value = decodeInternal(dataBuffer, charset);
+		DataBufferUtils.release(dataBuffer);
+		LogFormatUtils.traceDebug(logger, traceOn -> {
+			String formatted = LogFormatUtils.formatValue(value, !traceOn);
+			return Hints.getLogPrefix(hints) + "Decoded " + formatted;
+		});
+		return value;
 	}
 
 	private Charset getCharset(@Nullable MimeType mimeType) {

spring-core/src/main/java/org/springframework/core/io/AbstractFileResolvingResource.java

@@ -361,22 +361,12 @@ public abstract class AbstractFileResolvingResource extends AbstractResource {
 	 * @throws IOException if thrown from URLConnection methods
 	 */
 	protected void customizeConnection(URLConnection con) throws IOException {
-		useCachesIfNecessary(con);
+		ResourceUtils.useCachesIfNecessary(con);
 		if (con instanceof HttpURLConnection httpCon) {
 			customizeConnection(httpCon);
 		}
 	}
 
-	/**
-	 * Apply {@link URLConnection#setUseCaches useCaches} if necessary.
-	 * @param con the URLConnection to customize
-	 * @since 6.2.10
-	 * @see ResourceUtils#useCachesIfNecessary(URLConnection)
-	 */
-	void useCachesIfNecessary(URLConnection con) {
-		ResourceUtils.useCachesIfNecessary(con);
-	}
-
 	/**
 	 * Customize the given {@link HttpURLConnection} before fetching the resource.
 	 * <p>Can be overridden in subclasses for configuring request headers and timeouts.

spring-core/src/main/java/org/springframework/core/io/FileUrlResource.java

@@ -109,9 +109,7 @@ public class FileUrlResource extends UrlResource implements WritableResource {
 
 	@Override
 	public Resource createRelative(String relativePath) throws MalformedURLException {
-		FileUrlResource resource = new FileUrlResource(createRelativeURL(relativePath));
-		resource.useCaches = this.useCaches;
-		return resource;
+		return new FileUrlResource(createRelativeURL(relativePath));
 	}
 
 }

spring-core/src/main/java/org/springframework/core/io/Resource.java

@@ -117,10 +117,8 @@ public interface Resource extends InputStreamSource {
 
 	/**
 	 * Return a File handle for this resource.
-	 * <p>Note: This only works for files in the default file system.
-	 * @throws UnsupportedOperationException if the resource is a file but cannot be
-	 * exposed as a {@code java.io.File}; an alternative to {@code FileNotFoundException}
-	 * @throws java.io.FileNotFoundException if the resource cannot be resolved as a file
+	 * @throws java.io.FileNotFoundException if the resource cannot be resolved as
+	 * absolute file path, i.e. if the resource is not available in a file system
 	 * @throws IOException in case of general resolution/reading failures
 	 * @see #getInputStream()
 	 */

spring-core/src/main/java/org/springframework/core/io/UrlResource.java

@@ -67,12 +67,6 @@ public class UrlResource extends AbstractFileResolvingResource {
 	@Nullable
 	private volatile String cleanedUrl;
 
-	/**
-	 * Whether to use URLConnection caches ({@code null} means default).
-	 */
-	@Nullable
-	volatile Boolean useCaches;
-
 
 	/**
 	 * Create a new {@code UrlResource} based on the given URL object.
@@ -222,22 +216,11 @@ public class UrlResource extends AbstractFileResolvingResource {
 		return cleanedUrl;
 	}
 
-	/**
-	 * Set an explicit flag for {@link URLConnection#setUseCaches},
-	 * to be applied for any {@link URLConnection} operation in this resource.
-	 * <p>By default, caching will be applied only to jar resources.
-	 * An explicit {@code true} flag applies caching to all resources, whereas an
-	 * explicit {@code false} flag turns off caching for jar resources as well.
-	 * @since 6.2.10
-	 * @see ResourceUtils#useCachesIfNecessary
-	 */
-	public void setUseCaches(boolean useCaches) {
-		this.useCaches = useCaches;
-	}
-
 
 	/**
 	 * This implementation opens an InputStream for the given URL.
+	 * <p>It sets the {@code useCaches} flag to {@code false},
+	 * mainly to avoid jar file locking on Windows.
 	 * @see java.net.URL#openConnection()
 	 * @see java.net.URLConnection#setUseCaches(boolean)
 	 * @see java.net.URLConnection#getInputStream()
@@ -268,17 +251,6 @@ public class UrlResource extends AbstractFileResolvingResource {
 		}
 	}
 
-	@Override
-	void useCachesIfNecessary(URLConnection con) {
-		Boolean useCaches = this.useCaches;
-		if (useCaches != null) {
-			con.setUseCaches(useCaches);
-		}
-		else {
-			super.useCachesIfNecessary(con);
-		}
-	}
-
 	/**
 	 * This implementation returns the underlying URL reference.
 	 */
@@ -333,9 +305,7 @@ public class UrlResource extends AbstractFileResolvingResource {
 	 */
 	@Override
 	public Resource createRelative(String relativePath) throws MalformedURLException {
-		UrlResource resource = new UrlResource(createRelativeURL(relativePath));
-		resource.useCaches = this.useCaches;
-		return resource;
+		return new UrlResource(createRelativeURL(relativePath));
 	}
 
 	/**

spring-core/src/main/java/org/springframework/core/io/buffer/DataBuffer.java

@@ -36,9 +36,8 @@ import org.springframework.util.Assert;
  * <p>{@code DataBuffer}s has a separate {@linkplain #readPosition() read} and
  * {@linkplain #writePosition() write} position, as opposed to {@code ByteBuffer}'s
  * single {@linkplain ByteBuffer#position() position}. As such, the {@code DataBuffer}
- * does not require a {@linkplain ByteBuffer#flip() flip} to read after writing.
- * In general, the following invariant holds for the read and write positions,
- * and the capacity:
+ * does not require a {@linkplain ByteBuffer#flip() flip} to read after writing. In general,
+ * the following invariant holds for the read and write positions, and the capacity:
  *
  * <blockquote>
  *     {@code 0} {@code <=}
@@ -47,13 +46,12 @@ import org.springframework.util.Assert;
  *     <i>capacity</i>
  * </blockquote>
  *
- * <p>The {@linkplain #capacity() capacity} of a {@code DataBuffer} is expanded on
- * demand, similar to {@code StringBuilder}.
+ * <p>The {@linkplain #capacity() capacity} of a {@code DataBuffer} is expanded on demand,
+ * similar to {@code StringBuilder}.
  *
- * <p>The main purpose of the {@code DataBuffer} abstraction is to provide a
- * convenient wrapper around {@link ByteBuffer} which is similar to Netty's
- * {@link io.netty.buffer.ByteBuf} but can also be used on non-Netty platforms
- * (i.e. Servlet containers).
+ * <p>The main purpose of the {@code DataBuffer} abstraction is to provide a convenient wrapper
+ * around {@link ByteBuffer} which is similar to Netty's {@link io.netty.buffer.ByteBuf} but
+ * can also be used on non-Netty platforms (i.e. Servlet containers).
  *
  * @author Arjen Poutsma
  * @author Brian Clozel
@@ -115,8 +113,8 @@ public interface DataBuffer {
 	 * the current capacity, it will be expanded.
 	 * @param capacity the new capacity
 	 * @return this buffer
-	 * @deprecated as of 6.0, in favor of {@link #ensureWritable(int)}
-	 * which has different semantics
+	 * @deprecated as of 6.0, in favor of {@link #ensureWritable(int)}, which
+	 * has different semantics
 	 */
 	@Deprecated(since = "6.0")
 	DataBuffer capacity(int capacity);
@@ -188,28 +186,6 @@ public interface DataBuffer {
 	 */
 	byte getByte(int index);
 
-	/**
-	 * Process a range of bytes from the current buffer using a
-	 * {@link ByteProcessor}.
-	 * @param index the index at which the processing will start
-	 * @param length the maximum number of bytes to be processed
-	 * @param processor the processor that consumes bytes
-	 * @return the position that was reached when processing was stopped,
-	 * or {@code -1} if the entire byte range was processed.
-	 * @throws IndexOutOfBoundsException when {@code index} is out of bounds
-	 * @since 6.2.12
-	 */
-	default int forEachByte(int index, int length, ByteProcessor processor) {
-		Assert.isTrue(length >= 0, "Length must be >= 0");
-		for (int position = index; position < index + length; position++) {
-			byte b = getByte(position);
-			if(!processor.process(b)) {
-				return position;
-			}
-		}
-		return -1;
-	}
-
 	/**
 	 * Read a single byte from the current reading position from this data buffer.
 	 * @return the byte at this buffer's current reading position
@@ -242,16 +218,16 @@ public interface DataBuffer {
 	DataBuffer write(byte b);
 
 	/**
-	 * Write the given source into this buffer, starting at the current writing
-	 * position of this buffer.
+	 * Write the given source into this buffer, starting at the current writing position
+	 * of this buffer.
 	 * @param source the bytes to be written into this buffer
 	 * @return this buffer
 	 */
 	DataBuffer write(byte[] source);
 
 	/**
-	 * Write at most {@code length} bytes of the given source into this buffer,
-	 * starting at the current writing position of this buffer.
+	 * Write at most {@code length} bytes of the given source into this buffer, starting
+	 * at the current writing position of this buffer.
 	 * @param source the bytes to be written into this buffer
 	 * @param offset the index within {@code source} to start writing from
 	 * @param length the maximum number of bytes to be written from {@code source}
@@ -260,8 +236,8 @@ public interface DataBuffer {
 	DataBuffer write(byte[] source, int offset, int length);
 
 	/**
-	 * Write one or more {@code DataBuffer}s to this buffer, starting at the
-	 * current writing position. It is the responsibility of the caller to
+	 * Write one or more {@code DataBuffer}s to this buffer, starting at the current
+	 * writing position. It is the responsibility of the caller to
 	 * {@linkplain DataBufferUtils#release(DataBuffer) release} the given data buffers.
 	 * @param buffers the byte buffers to write into this buffer
 	 * @return this buffer
@@ -269,8 +245,8 @@ public interface DataBuffer {
 	DataBuffer write(DataBuffer... buffers);
 
 	/**
-	 * Write one or more {@link ByteBuffer} to this buffer, starting at the
-	 * current writing position.
+	 * Write one or more {@link ByteBuffer} to this buffer, starting at the current
+	 * writing position.
 	 * @param buffers the byte buffers to write into this buffer
 	 * @return this buffer
 	 */
@@ -323,37 +299,36 @@ public interface DataBuffer {
 	}
 
 	/**
-	 * Create a new {@code DataBuffer} whose contents is a shared subsequence
-	 * of this data buffer's content.  Data between this data buffer and the
-	 * returned buffer is shared; though changes in the returned buffer's
-	 * position will not be reflected in the reading nor writing position
-	 * of this data buffer.
+	 * Create a new {@code DataBuffer} whose contents is a shared subsequence of this
+	 * data buffer's content.  Data between this data buffer and the returned buffer is
+	 * shared; though changes in the returned buffer's position will not be reflected
+	 * in the reading nor writing position of this data buffer.
 	 * <p><strong>Note</strong> that this method will <strong>not</strong> call
-	 * {@link DataBufferUtils#retain(DataBuffer)} on the resulting slice:
-	 * the reference count will not be increased.
+	 * {@link DataBufferUtils#retain(DataBuffer)} on the resulting slice: the reference
+	 * count will not be increased.
 	 * @param index the index at which to start the slice
 	 * @param length the length of the slice
 	 * @return the specified slice of this data buffer
-	 * @deprecated as of 6.0, in favor of {@link #split(int)}
-	 * which has different semantics
+	 * @deprecated as of 6.0, in favor of {@link #split(int)}, which
+	 * has different semantics
 	 */
 	@Deprecated(since = "6.0")
 	DataBuffer slice(int index, int length);
 
 	/**
-	 * Create a new {@code DataBuffer} whose contents is a shared, retained subsequence
-	 * of this data buffer's content.  Data between this data buffer and the returned
-	 * buffer is shared; though changes in the returned buffer's position will not be
-	 * reflected in the reading nor writing position of this data buffer.
+	 * Create a new {@code DataBuffer} whose contents is a shared, retained subsequence of this
+	 * data buffer's content.  Data between this data buffer and the returned buffer is
+	 * shared; though changes in the returned buffer's position will not be reflected
+	 * in the reading nor writing position of this data buffer.
 	 * <p><strong>Note</strong> that unlike {@link #slice(int, int)}, this method
-	 * <strong>will</strong> call {@link DataBufferUtils#retain(DataBuffer)}
-	 * (or equivalent) on the resulting slice.
+	 * <strong>will</strong> call {@link DataBufferUtils#retain(DataBuffer)} (or equivalent) on the
+	 * resulting slice.
 	 * @param index the index at which to start the slice
 	 * @param length the length of the slice
 	 * @return the specified, retained slice of this data buffer
 	 * @since 5.2
-	 * @deprecated as of 6.0, in favor of {@link #split(int)}
-	 * which has different semantics
+	 * @deprecated as of 6.0, in favor of {@link #split(int)}, which
+	 * has different semantics
 	 */
 	@Deprecated(since = "6.0")
 	default DataBuffer retainedSlice(int index, int length) {
@@ -362,10 +337,12 @@ public interface DataBuffer {
 
 	/**
 	 * Splits this data buffer into two at the given index.
+	 *
 	 * <p>Data that precedes the {@code index} will be returned in a new buffer,
 	 * while this buffer will contain data that follows after {@code index}.
 	 * Memory between the two buffers is shared, but independent and cannot
 	 * overlap (unlike {@link #slice(int, int) slice}).
+	 *
 	 * <p>The {@linkplain #readPosition() read} and
 	 * {@linkplain #writePosition() write} position of the returned buffer are
 	 * truncated to fit within the buffers {@linkplain #capacity() capacity} if
@@ -385,22 +362,22 @@ public interface DataBuffer {
 	 * will not be reflected in the reading nor writing position of this data buffer.
 	 * @return this data buffer as a byte buffer
 	 * @deprecated as of 6.0, in favor of {@link #toByteBuffer(ByteBuffer)},
-	 * {@link #readableByteBuffers()} or {@link #writableByteBuffers()}
+	 * {@link #readableByteBuffers()}, or {@link #writableByteBuffers()}.
 	 */
 	@Deprecated(since = "6.0")
 	ByteBuffer asByteBuffer();
 
 	/**
-	 * Expose a subsequence of this buffer's bytes as a {@link ByteBuffer}. Data
-	 * between this {@code DataBuffer} and the returned {@code ByteBuffer} is shared;
-	 * though changes in the returned buffer's {@linkplain ByteBuffer#position() position}
+	 * Expose a subsequence of this buffer's bytes as a {@link ByteBuffer}. Data between
+	 * this {@code DataBuffer} and the returned {@code ByteBuffer} is shared; though
+	 * changes in the returned buffer's {@linkplain ByteBuffer#position() position}
 	 * will not be reflected in the reading nor writing position of this data buffer.
 	 * @param index the index at which to start the byte buffer
 	 * @param length the length of the returned byte buffer
 	 * @return this data buffer as a byte buffer
 	 * @since 5.0.1
 	 * @deprecated as of 6.0, in favor of {@link #toByteBuffer(int, ByteBuffer, int, int)},
-	 * {@link #readableByteBuffers()} or {@link #writableByteBuffers()}
+	 * {@link #readableByteBuffers()}, or {@link #writableByteBuffers()}.
 	 */
 	@Deprecated(since = "6.0")
 	ByteBuffer asByteBuffer(int index, int length);
@@ -554,22 +531,4 @@ public interface DataBuffer {
 		void close();
 	}
 
-
-	/**
-	 * Process a range of bytes one by one.
-	 * @since 6.2.12
-	 */
-	@FunctionalInterface
-	interface ByteProcessor {
-
-		/**
-		 * Process the given {@code byte} and indicate whether processing
-		 * should continue further.
-		 * @param b a byte from the {@link DataBuffer}
-		 * @return {@code true} if processing should continue,
-		 * or {@code false} if processing should stop at this element
-		 */
-		boolean process(byte b);
-	}
-
 }

spring-core/src/main/java/org/springframework/core/io/buffer/DataBufferLimitException.java

@@ -31,19 +31,8 @@ package org.springframework.core.io.buffer;
 public class DataBufferLimitException extends IllegalStateException {
 
 
-	/**
-	 * Create an instance with the given message.
-	 */
 	public DataBufferLimitException(String message) {
 		super(message);
 	}
 
-	/**
-	 * Create an instance with a message and a cause, e.g. {@link OutOfMemoryError}.
-	 * @since 6.2.12
-	 */
-	public DataBufferLimitException(String message, Throwable cause) {
-		super(message, cause);
-	}
-
 }

spring-core/src/main/java/org/springframework/core/io/buffer/DataBufferUtils.java

@@ -16,6 +16,7 @@
 
 package org.springframework.core.io.buffer;
 
+import java.io.File;
 import java.io.IOException;
 import java.io.InputStream;
 import java.io.OutputStream;
@@ -222,17 +223,17 @@ public abstract class DataBufferUtils {
 
 		try {
 			if (resource.isFile()) {
-				Path filePath = resource.getFile().toPath();
+				File file = resource.getFile();
 				return readAsynchronousFileChannel(
-						() -> AsynchronousFileChannel.open(filePath, StandardOpenOption.READ),
+						() -> AsynchronousFileChannel.open(file.toPath(), StandardOpenOption.READ),
 						position, bufferFactory, bufferSize);
 			}
 		}
-		catch (IOException | UnsupportedOperationException ignore) {
+		catch (IOException ignore) {
 			// fallback to resource.readableChannel(), below
 		}
 		Flux<DataBuffer> result = readByteChannel(resource::readableChannel, bufferFactory, bufferSize);
-		return (position == 0 ? result : skipUntilByteCount(result, position));
+		return position == 0 ? result : skipUntilByteCount(result, position);
 	}
 
 
@@ -832,9 +833,13 @@ public abstract class DataBufferUtils {
 
 		@Override
 		public int match(DataBuffer dataBuffer) {
-			int start = dataBuffer.readPosition();
-			int end = dataBuffer.writePosition();
-			return dataBuffer.forEachByte(start, end - start, b -> !this.match(b));
+			for (int pos = dataBuffer.readPosition(); pos < dataBuffer.writePosition(); pos++) {
+				byte b = dataBuffer.getByte(pos);
+				if (match(b)) {
+					return pos;
+				}
+			}
+			return -1;
 		}
 
 		@Override
@@ -877,13 +882,14 @@ public abstract class DataBufferUtils {
 
 		@Override
 		public int match(DataBuffer dataBuffer) {
-			int start = dataBuffer.readPosition();
-			int end = dataBuffer.writePosition();
-			int matchPosition = dataBuffer.forEachByte(start, end - start, b -> !this.match(b));
-			if (matchPosition != -1) {
-				reset();
+			for (int pos = dataBuffer.readPosition(); pos < dataBuffer.writePosition(); pos++) {
+				byte b = dataBuffer.getByte(pos);
+				if (match(b)) {
+					reset();
+					return pos;
+				}
 			}
-			return matchPosition;
+			return -1;
 		}
 
 		@Override

spring-core/src/main/java/org/springframework/core/io/buffer/JettyDataBuffer.java

@@ -55,6 +55,7 @@ public final class JettyDataBuffer implements PooledDataBuffer {
 		this.bufferFactory = bufferFactory;
 		this.delegate = delegate;
 		this.chunk = chunk;
+		this.chunk.retain();
 	}
 
 	JettyDataBuffer(JettyDataBufferFactory bufferFactory, DefaultDataBuffer delegate) {

spring-core/src/main/java/org/springframework/core/io/buffer/NettyDataBuffer.java

@@ -129,11 +129,6 @@ public class NettyDataBuffer implements PooledDataBuffer {
 		return this.byteBuf.getByte(index);
 	}
 
-	@Override
-	public int forEachByte(int index, int length, ByteProcessor processor) {
-		return this.byteBuf.forEachByte(index, length, processor::process);
-	}
-
 	@Override
 	public int capacity() {
 		return this.byteBuf.capacity();
@@ -379,13 +374,7 @@ public class NettyDataBuffer implements PooledDataBuffer {
 
 	@Override
 	public String toString() {
-		try {
-			return this.byteBuf.toString();
-		}
-		catch (OutOfMemoryError ex) {
-			throw new DataBufferLimitException(
-					"Failed to convert data buffer to string: " + ex.getMessage(), ex);
-		}
+		return this.byteBuf.toString();
 	}
 
 

spring-core/src/main/java/org/springframework/core/io/support/PathMatchingResourcePatternResolver.java

@@ -41,6 +41,7 @@ import java.nio.file.Path;
 import java.util.Collections;
 import java.util.Enumeration;
 import java.util.HashSet;
+import java.util.Iterator;
 import java.util.LinkedHashSet;
 import java.util.Map;
 import java.util.NavigableSet;
@@ -259,8 +260,7 @@ public class PathMatchingResourcePatternResolver implements ResourcePatternResol
 
 	private PathMatcher pathMatcher = new AntPathMatcher();
 
-	@Nullable
-	private Boolean useCaches;
+	private boolean useCaches = true;
 
 	private final Map<String, Resource[]> rootDirCache = new ConcurrentHashMap<>();
 
@@ -342,12 +342,10 @@ public class PathMatchingResourcePatternResolver implements ResourcePatternResol
 	 * the {@link JarURLConnection} level as well as within this resolver instance.
 	 * <p>Note that {@link JarURLConnection#setDefaultUseCaches} can be turned off
 	 * independently. This resolver-level setting is designed to only enforce
-	 * {@code JarURLConnection#setUseCaches(true/false)} if necessary but otherwise
-	 * leaves the JVM-level default in place (if this setter has not been called).
-	 * <p>As of 6.2.10, this setting propagates to {@link UrlResource#setUseCaches}.
+	 * {@code JarURLConnection#setUseCaches(false)} if necessary but otherwise
+	 * leaves the JVM-level default in place.
 	 * @since 6.1.19
 	 * @see JarURLConnection#setUseCaches
-	 * @see UrlResource#setUseCaches
 	 * @see #clearCache()
 	 */
 	public void setUseCaches(boolean useCaches) {
@@ -357,11 +355,7 @@ public class PathMatchingResourcePatternResolver implements ResourcePatternResol
 
 	@Override
 	public Resource getResource(String location) {
-		Resource resource = getResourceLoader().getResource(location);
-		if (this.useCaches != null && resource instanceof UrlResource urlResource) {
-			urlResource.setUseCaches(this.useCaches);
-		}
-		return resource;
+		return getResourceLoader().getResource(location);
 	}
 
 	@Override
@@ -394,7 +388,7 @@ public class PathMatchingResourcePatternResolver implements ResourcePatternResol
 			}
 			else {
 				// a single resource with the given name
-				return new Resource[] {getResource(locationPattern)};
+				return new Resource[] {getResourceLoader().getResource(locationPattern)};
 			}
 		}
 	}
@@ -479,27 +473,20 @@ public class PathMatchingResourcePatternResolver implements ResourcePatternResol
 			}
 		}
 		else {
-			UrlResource resource = null;
 			String urlString = url.toString();
 			String cleanedPath = StringUtils.cleanPath(urlString);
 			if (!cleanedPath.equals(urlString)) {
 				// Prefer cleaned URL, aligned with UrlResource#createRelative(String)
 				try {
 					// Retain original URL instance, potentially including custom URLStreamHandler.
-					resource = new UrlResource(new URL(url, cleanedPath));
+					return new UrlResource(new URL(url, cleanedPath));
 				}
 				catch (MalformedURLException ex) {
 					// Fallback to regular URL construction below...
 				}
 			}
 			// Retain original URL instance, potentially including custom URLStreamHandler.
-			if (resource == null) {
-				resource = new UrlResource(url);
-			}
-			if (this.useCaches != null) {
-				resource.setUseCaches(this.useCaches);
-			}
-			return resource;
+			return new UrlResource(url);
 		}
 	}
 
@@ -518,9 +505,6 @@ public class PathMatchingResourcePatternResolver implements ResourcePatternResol
 						UrlResource jarResource = (ResourceUtils.URL_PROTOCOL_JAR.equals(url.getProtocol()) ?
 								new UrlResource(url) :
 								new UrlResource(ResourceUtils.JAR_URL_PREFIX + url + ResourceUtils.JAR_URL_SEPARATOR));
-						if (this.useCaches != null) {
-							jarResource.setUseCaches(this.useCaches);
-						}
 						if (jarResource.exists()) {
 							result.add(jarResource);
 						}
@@ -572,7 +556,7 @@ public class PathMatchingResourcePatternResolver implements ResourcePatternResol
 		Set<ClassPathManifestEntry> entries = this.manifestEntriesCache;
 		if (entries == null) {
 			entries = getClassPathManifestEntries();
-			if (this.useCaches == null || this.useCaches) {
+			if (this.useCaches) {
 				this.manifestEntriesCache = entries;
 			}
 		}
@@ -593,7 +577,7 @@ public class PathMatchingResourcePatternResolver implements ResourcePatternResol
 				try {
 					File jar = new File(path).getAbsoluteFile();
 					if (jar.isFile() && seen.add(jar)) {
-						manifestEntries.add(ClassPathManifestEntry.of(jar, this.useCaches));
+						manifestEntries.add(ClassPathManifestEntry.of(jar));
 						manifestEntries.addAll(getClassPathManifestEntriesFromJar(jar));
 					}
 				}
@@ -632,7 +616,7 @@ public class PathMatchingResourcePatternResolver implements ResourcePatternResol
 					}
 					File candidate = new File(parent, path);
 					if (candidate.isFile() && candidate.getCanonicalPath().contains(parent.getCanonicalPath())) {
-						manifestEntries.add(ClassPathManifestEntry.of(candidate, this.useCaches));
+						manifestEntries.add(ClassPathManifestEntry.of(candidate));
 					}
 				}
 			}
@@ -726,7 +710,7 @@ public class PathMatchingResourcePatternResolver implements ResourcePatternResol
 			if (rootDirResources == null) {
 				// Lookup for specific directory, creating a cache entry for it.
 				rootDirResources = getResources(rootDirPath);
-				if (this.useCaches == null || this.useCaches) {
+				if (this.useCaches) {
 					this.rootDirCache.put(rootDirPath, rootDirResources);
 				}
 			}
@@ -745,11 +729,7 @@ public class PathMatchingResourcePatternResolver implements ResourcePatternResol
 				if (resolvedUrl != null) {
 					rootDirUrl = resolvedUrl;
 				}
-				UrlResource urlResource = new UrlResource(rootDirUrl);
-				if (this.useCaches != null) {
-					urlResource.setUseCaches(this.useCaches);
-				}
-				rootDirResource = urlResource;
+				rootDirResource = new UrlResource(rootDirUrl);
 			}
 			if (rootDirUrl.getProtocol().startsWith(ResourceUtils.URL_PROTOCOL_VFS)) {
 				result.addAll(VfsResourceMatchingDelegate.findMatchingResources(rootDirUrl, subPattern, getPathMatcher()));
@@ -885,8 +865,8 @@ public class PathMatchingResourcePatternResolver implements ResourcePatternResol
 
 		if (con instanceof JarURLConnection jarCon) {
 			// Should usually be the case for traditional JAR files.
-			if (this.useCaches != null) {
-				jarCon.setUseCaches(this.useCaches);
+			if (!this.useCaches) {
+				jarCon.setUseCaches(false);
 			}
 			try {
 				jarFile = jarCon.getJarFile();
@@ -936,10 +916,14 @@ public class PathMatchingResourcePatternResolver implements ResourcePatternResol
 			}
 			Set<Resource> result = new LinkedHashSet<>(64);
 			NavigableSet<String> entriesCache = new TreeSet<>();
-			for (Enumeration<JarEntry> entries = jarFile.entries(); entries.hasMoreElements();) {
-				entriesCache.add(entries.nextElement().getName());
-			}
-			for (String entryPath : entriesCache) {
+			Iterator<String> entryIterator = jarFile.stream().map(JarEntry::getName).sorted().iterator();
+			while (entryIterator.hasNext()) {
+				String entryPath = entryIterator.next();
+				int entrySeparatorIndex = entryPath.indexOf(ResourceUtils.JAR_URL_SEPARATOR);
+				if (entrySeparatorIndex >= 0) {
+					entryPath = entryPath.substring(entrySeparatorIndex + ResourceUtils.JAR_URL_SEPARATOR.length());
+				}
+				entriesCache.add(entryPath);
 				if (entryPath.startsWith(rootEntryPath)) {
 					String relativePath = entryPath.substring(rootEntryPath.length());
 					if (getPathMatcher().match(subPattern, relativePath)) {
@@ -947,7 +931,7 @@ public class PathMatchingResourcePatternResolver implements ResourcePatternResol
 					}
 				}
 			}
-			if (this.useCaches == null || this.useCaches) {
+			if (this.useCaches) {
 				// Cache jar entries in TreeSet for efficient searching on re-encounter.
 				this.jarEntriesCache.put(jarFileUrl, entriesCache);
 			}
@@ -1273,10 +1257,10 @@ public class PathMatchingResourcePatternResolver implements ResourcePatternResol
 
 		private static final String JARFILE_URL_PREFIX = ResourceUtils.JAR_URL_PREFIX + ResourceUtils.FILE_URL_PREFIX;
 
-		static ClassPathManifestEntry of(File file, @Nullable Boolean useCaches) throws MalformedURLException {
+		static ClassPathManifestEntry of(File file) throws MalformedURLException {
 			String path = fixPath(file.getAbsolutePath());
-			Resource resource = asJarFileResource(path, useCaches);
-			Resource alternative = createAlternative(path, useCaches);
+			Resource resource = asJarFileResource(path);
+			Resource alternative = createAlternative(path);
 			return new ClassPathManifestEntry(resource, alternative);
 		}
 
@@ -1297,22 +1281,18 @@ public class PathMatchingResourcePatternResolver implements ResourcePatternResol
 		 * @return the alternative form or {@code null}
 		 */
 		@Nullable
-		private static Resource createAlternative(String path, @Nullable Boolean useCaches) {
+		private static Resource createAlternative(String path) {
 			try {
 				String alternativePath = path.startsWith("/") ? path.substring(1) : "/" + path;
-				return asJarFileResource(alternativePath, useCaches);
+				return asJarFileResource(alternativePath);
 			}
 			catch (MalformedURLException ex) {
 				return null;
 			}
 		}
 
-		private static Resource asJarFileResource(String path, @Nullable Boolean useCaches) throws MalformedURLException {
-			UrlResource resource = new UrlResource(JARFILE_URL_PREFIX + path + ResourceUtils.JAR_URL_SEPARATOR);
-			if (useCaches != null) {
-				resource.setUseCaches(useCaches);
-			}
-			return resource;
+		private static Resource asJarFileResource(String path) throws MalformedURLException {
+			return new UrlResource(JARFILE_URL_PREFIX + path + ResourceUtils.JAR_URL_SEPARATOR);
 		}
 	}
 

spring-core/src/main/java/org/springframework/core/task/SimpleAsyncTaskExecutor.java

@@ -92,8 +92,6 @@ public class SimpleAsyncTaskExecutor extends CustomizableThreadCreator
 	@Nullable
 	private Set<Thread> activeThreads;
 
-	private boolean cancelRemainingTasksOnClose = false;
-
 	private boolean rejectTasksWhenLimitReached = false;
 
 	private volatile boolean active = true;
@@ -186,33 +184,12 @@ public class SimpleAsyncTaskExecutor extends CustomizableThreadCreator
 	 * @param timeout the timeout in milliseconds
 	 * @since 6.1
 	 * @see #close()
-	 * @see #setCancelRemainingTasksOnClose
 	 * @see org.springframework.scheduling.concurrent.ExecutorConfigurationSupport#setAwaitTerminationMillis
 	 */
 	public void setTaskTerminationTimeout(long timeout) {
 		Assert.isTrue(timeout >= 0, "Timeout value must be >=0");
 		this.taskTerminationTimeout = timeout;
-		trackActiveThreadsIfNecessary();
-	}
-
-	/**
-	 * Specify whether to cancel remaining tasks on close: that is, whether to
-	 * interrupt any active threads at the time of the {@link #close()} call.
-	 * <p>The default is {@code false}, not tracking active threads at all or
-	 * just interrupting any remaining threads that still have not finished after
-	 * the specified {@link #setTaskTerminationTimeout taskTerminationTimeout}.
-	 * Switch this to {@code true} for immediate interruption on close, either in
-	 * combination with a subsequent termination timeout or without any waiting
-	 * at all, depending on whether a {@code taskTerminationTimeout} has been
-	 * specified as well.
-	 * @since 6.2.11
-	 * @see #close()
-	 * @see #setTaskTerminationTimeout
-	 * @see org.springframework.scheduling.concurrent.ExecutorConfigurationSupport#setWaitForTasksToCompleteOnShutdown
-	 */
-	public void setCancelRemainingTasksOnClose(boolean cancelRemainingTasksOnClose) {
-		this.cancelRemainingTasksOnClose = cancelRemainingTasksOnClose;
-		trackActiveThreadsIfNecessary();
+		this.activeThreads = (timeout > 0 ? ConcurrentHashMap.newKeySet() : null);
 	}
 
 	/**
@@ -272,15 +249,6 @@ public class SimpleAsyncTaskExecutor extends CustomizableThreadCreator
 		return this.active;
 	}
 
-	/**
-	 * Track active threads only when a task termination timeout has been
-	 * specified or interruption of remaining threads has been requested.
-	 */
-	private void trackActiveThreadsIfNecessary() {
-		this.activeThreads = (this.taskTerminationTimeout > 0 || this.cancelRemainingTasksOnClose ?
-				ConcurrentHashMap.newKeySet() : null);
-	}
-
 
 	/**
 	 * Executes the given task, within a concurrency throttle
@@ -385,7 +353,7 @@ public class SimpleAsyncTaskExecutor extends CustomizableThreadCreator
 	}
 
 	/**
-	 * This close method tracks the termination of active threads if a concrete
+	 * This close methods tracks the termination of active threads if a concrete
 	 * {@link #setTaskTerminationTimeout task termination timeout} has been set.
 	 * Otherwise, it is not necessary to close this executor.
 	 * @since 6.1
@@ -396,24 +364,15 @@ public class SimpleAsyncTaskExecutor extends CustomizableThreadCreator
 			this.active = false;
 			Set<Thread> threads = this.activeThreads;
 			if (threads != null) {
-				if (this.cancelRemainingTasksOnClose) {
-					// Early interrupt for remaining tasks on close
-					threads.forEach(Thread::interrupt);
-				}
-				if (this.taskTerminationTimeout > 0) {
-					synchronized (threads) {
-						try {
-							if (!threads.isEmpty()) {
-								threads.wait(this.taskTerminationTimeout);
-							}
-						}
-						catch (InterruptedException ex) {
-							Thread.currentThread().interrupt();
+				threads.forEach(Thread::interrupt);
+				synchronized (threads) {
+					try {
+						if (!threads.isEmpty()) {
+							threads.wait(this.taskTerminationTimeout);
 						}
 					}
-					if (!this.cancelRemainingTasksOnClose) {
-						// Late interrupt for remaining tasks after timeout
-						threads.forEach(Thread::interrupt);
+					catch (InterruptedException ex) {
+						Thread.currentThread().interrupt();
 					}
 				}
 			}

spring-core/src/main/java/org/springframework/core/task/SyncTaskExecutor.java

@@ -22,7 +22,9 @@ import org.springframework.util.Assert;
 
 /**
  * {@link TaskExecutor} implementation that executes each task <i>synchronously</i>
- * in the calling thread. Mainly intended for testing scenarios.
+ * in the calling thread.
+ *
+ * <p>Mainly intended for testing scenarios.
  *
  * <p>Execution in the calling thread does have the advantage of participating
  * in its thread context, for example the thread context class loader or the
@@ -38,13 +40,13 @@ import org.springframework.util.Assert;
 public class SyncTaskExecutor implements TaskExecutor, Serializable {
 
 	/**
-	 * Execute the given {@code task} synchronously, through direct
-	 * invocation of its {@link Runnable#run() run()} method.
-	 * @throws RuntimeException if propagated from the given {@code Runnable}
+	 * Executes the given {@code task} synchronously, through direct
+	 * invocation of it's {@link Runnable#run() run()} method.
+	 * @throws IllegalArgumentException if the given {@code task} is {@code null}
 	 */
 	@Override
 	public void execute(Runnable task) {
-		Assert.notNull(task, "Task must not be null");
+		Assert.notNull(task, "Runnable must not be null");
 		task.run();
 	}
 

spring-core/src/main/java/org/springframework/lang/Contract.java

@@ -51,8 +51,7 @@ import java.lang.annotation.Target;
  * <p>The additional return values denote the following:
  * <ul>
  * <li>{@code fail} - the method throws an exception, if the arguments satisfy argument constraints
- * <li>{@code new} - the method returns a non-null new object which is distinct from any other object existing in the heap prior to method execution.
- * If the method has no visible side effects, then we can be sure that the new object is not stored to any field/array and will be lost if the method's return value is not used.
+ * <li>{@code new} - the method returns a non-null new object which is distinct from any other object existing in the heap prior to method execution. If method is also pure, then we can be sure that the new object is not stored to any field/array and will be lost if method return value is not used.
  * <li>{@code this} - the method returns its qualifier value (not applicable for static methods)
  * <li>{@code param1, param2, ...} - the method returns its first (second, ...) parameter value
  * </ul>

spring-core/src/main/java/org/springframework/util/ClassUtils.java

@@ -1483,8 +1483,8 @@ public abstract class ClassUtils {
 	}
 
 	/**
-	 * Get the closest publicly accessible (and exported) method in the supplied method's type
-	 * hierarchy that has a method signature equivalent to the supplied method, if possible.
+	 * Get the highest publicly accessible method in the supplied method's type hierarchy that
+	 * has a method signature equivalent to the supplied method, if possible.
 	 * <p>Otherwise, this method recursively searches the class hierarchy and implemented
 	 * interfaces for an equivalent method that is {@code public} and declared in a
 	 * {@code public} type.
@@ -1507,21 +1507,18 @@ public abstract class ClassUtils {
 	 * @see #getMostSpecificMethod(Method, Class)
 	 */
 	public static Method getPubliclyAccessibleMethodIfPossible(Method method, @Nullable Class<?> targetClass) {
-		Class<?> declaringClass = method.getDeclaringClass();
-		// If the method is not public or its declaring class is public and exported already,
-		// we can abort the search immediately (avoiding reflection as well as cache access).
-		if (!Modifier.isPublic(method.getModifiers()) || (Modifier.isPublic(declaringClass.getModifiers()) &&
-				declaringClass.getModule().isExported(declaringClass.getPackageName(), ClassUtils.class.getModule()))) {
+		// If the method is not public, we can abort the search immediately.
+		if (!Modifier.isPublic(method.getModifiers())) {
 			return method;
 		}
 
 		Method interfaceMethod = getInterfaceMethodIfPossible(method, targetClass, true);
 		// If we found a method in a public interface, return the interface method.
-		if (interfaceMethod != method && interfaceMethod.getDeclaringClass().getModule().isExported(
-				interfaceMethod.getDeclaringClass().getPackageName(), ClassUtils.class.getModule())) {
+		if (interfaceMethod != method) {
 			return interfaceMethod;
 		}
 
+		Class<?> declaringClass = method.getDeclaringClass();
 		// Bypass cache for java.lang.Object unless it is actually an overridable method declared there.
 		if (declaringClass.getSuperclass() == Object.class && !ReflectionUtils.isObjectMethod(method)) {
 			return method;
@@ -1536,20 +1533,19 @@ public abstract class ClassUtils {
 	private static Method findPubliclyAccessibleMethodIfPossible(
 			String methodName, Class<?>[] parameterTypes, Class<?> declaringClass) {
 
+		Method result = null;
 		Class<?> current = declaringClass.getSuperclass();
 		while (current != null) {
 			Method method = getMethodOrNull(current, methodName, parameterTypes);
 			if (method == null) {
 				break;
 			}
-			if (Modifier.isPublic(method.getDeclaringClass().getModifiers()) &&
-					method.getDeclaringClass().getModule().isExported(
-							method.getDeclaringClass().getPackageName(), ClassUtils.class.getModule())) {
-				return method;
+			if (Modifier.isPublic(method.getDeclaringClass().getModifiers())) {
+				result = method;
 			}
 			current = method.getDeclaringClass().getSuperclass();
 		}
-		return null;
+		return result;
 	}
 
 	/**

spring-core/src/main/java/org/springframework/util/FileSystemUtils.java

@@ -86,12 +86,13 @@ public abstract class FileSystemUtils {
 
 		Files.walkFileTree(root, new SimpleFileVisitor<>() {
 			@Override
-			public FileVisitResult visitFile(Path file, BasicFileAttributes attr) throws IOException {
+			public FileVisitResult visitFile(Path file, BasicFileAttributes attrs) throws IOException {
 				Files.delete(file);
 				return FileVisitResult.CONTINUE;
 			}
+
 			@Override
-			public FileVisitResult postVisitDirectory(Path dir, IOException ex) throws IOException {
+			public FileVisitResult postVisitDirectory(Path dir, IOException exc) throws IOException {
 				Files.delete(dir);
 				return FileVisitResult.CONTINUE;
 			}
@@ -126,34 +127,19 @@ public abstract class FileSystemUtils {
 		BasicFileAttributes srcAttr = Files.readAttributes(src, BasicFileAttributes.class);
 
 		if (srcAttr.isDirectory()) {
-			if (src.getClass() == dest.getClass()) {  // dest.resolve(Path) only works for same Path type
-				Files.walkFileTree(src, EnumSet.of(FOLLOW_LINKS), Integer.MAX_VALUE, new SimpleFileVisitor<>() {
-					@Override
-					public FileVisitResult preVisitDirectory(Path dir, BasicFileAttributes attr) throws IOException {
-						Files.createDirectories(dest.resolve(src.relativize(dir)));
-						return FileVisitResult.CONTINUE;
-					}
-					@Override
-					public FileVisitResult visitFile(Path file, BasicFileAttributes attr) throws IOException {
-						Files.copy(file, dest.resolve(src.relativize(file)), StandardCopyOption.REPLACE_EXISTING);
-						return FileVisitResult.CONTINUE;
-					}
-				});
-			}
-			else {  // use dest.resolve(String) for different Path types
-				Files.walkFileTree(src, EnumSet.of(FOLLOW_LINKS), Integer.MAX_VALUE, new SimpleFileVisitor<>() {
-					@Override
-					public FileVisitResult preVisitDirectory(Path dir, BasicFileAttributes attr) throws IOException {
-						Files.createDirectories(dest.resolve(src.relativize(dir).toString()));
-						return FileVisitResult.CONTINUE;
-					}
-					@Override
-					public FileVisitResult visitFile(Path file, BasicFileAttributes attr) throws IOException {
-						Files.copy(file, dest.resolve(src.relativize(file).toString()), StandardCopyOption.REPLACE_EXISTING);
-						return FileVisitResult.CONTINUE;
-					}
-				});
-			}
+			Files.walkFileTree(src, EnumSet.of(FOLLOW_LINKS), Integer.MAX_VALUE, new SimpleFileVisitor<>() {
+				@Override
+				public FileVisitResult preVisitDirectory(Path dir, BasicFileAttributes attrs) throws IOException {
+					Files.createDirectories(dest.resolve(src.relativize(dir)));
+					return FileVisitResult.CONTINUE;
+				}
+
+				@Override
+				public FileVisitResult visitFile(Path file, BasicFileAttributes attrs) throws IOException {
+					Files.copy(file, dest.resolve(src.relativize(file)), StandardCopyOption.REPLACE_EXISTING);
+					return FileVisitResult.CONTINUE;
+				}
+			});
 		}
 		else if (srcAttr.isRegularFile()) {
 			Files.copy(src, dest);

spring-core/src/main/java/org/springframework/util/PlaceholderParser.java

@@ -18,6 +18,7 @@ package org.springframework.util;
 
 import java.util.ArrayList;
 import java.util.HashSet;
+import java.util.LinkedList;
 import java.util.List;
 import java.util.Map;
 import java.util.Set;
@@ -35,10 +36,10 @@ import org.springframework.util.PropertyPlaceholderHelper.PlaceholderResolver;
  * that can be resolved using a {@link PlaceholderResolver PlaceholderResolver},
  * <code>${</code> the prefix, and <code>}</code> the suffix.
  *
- * <p>A placeholder can also have a default value if its key does not represent
- * a known property. The default value is separated from the key using a
- * {@code separator}. For instance {@code ${name:John}} resolves to {@code John}
- * if the placeholder resolver does not provide a value for the {@code name}
+ * <p>A placeholder can also have a default value if its key does not represent a
+ * known property. The default value is separated from the key using a
+ * {@code separator}. For instance {@code ${name:John}} resolves to {@code John} if
+ * the placeholder resolver does not provide a value for the {@code name}
  * property.
  *
  * <p>Placeholders can also have a more complex structure, and the resolution of
@@ -49,14 +50,13 @@ import org.springframework.util.PropertyPlaceholderHelper.PlaceholderResolver;
  * must be rendered as is, the placeholder can be escaped using an {@code escape}
  * character. For instance {@code \${name}} resolves as {@code ${name}}.
  *
- * <p>The prefix, suffix, separator, and escape characters are configurable.
- * Only the prefix and suffix are mandatory, and the support for default values
- * or escaping is conditional on providing non-null values for them.
+ * <p>The prefix, suffix, separator, and escape characters are configurable. Only
+ * the prefix and suffix are mandatory, and the support for default values or
+ * escaping is conditional on providing non-null values for them.
  *
  * <p>This parser makes sure to resolves placeholders as lazily as possible.
  *
  * @author Stephane Nicoll
- * @author Juergen Hoeller
  * @since 6.2
  */
 final class PlaceholderParser {
@@ -120,47 +120,51 @@ final class PlaceholderParser {
 	 * @return the supplied value with placeholders replaced inline
 	 */
 	public String replacePlaceholders(String value, PlaceholderResolver placeholderResolver) {
-		List<Part> parts = parse(value, false);
-		if (parts == null) {
-			return value;
-		}
-		ParsedValue parsedValue = new ParsedValue(value, parts);
+		Assert.notNull(value, "'value' must not be null");
+		ParsedValue parsedValue = parse(value);
 		PartResolutionContext resolutionContext = new PartResolutionContext(placeholderResolver,
 				this.prefix, this.suffix, this.ignoreUnresolvablePlaceholders,
 				candidate -> parse(candidate, false));
 		return parsedValue.resolve(resolutionContext);
 	}
 
-	private @Nullable List<Part> parse(String value, boolean inPlaceholder) {
+	/**
+	 * Parse the specified value.
+	 * @param value the value containing the placeholders to be replaced
+	 * @return the different parts that have been identified
+	 */
+	ParsedValue parse(String value) {
+		List<Part> parts = parse(value, false);
+		return new ParsedValue(value, parts);
+	}
+
+	private List<Part> parse(String value, boolean inPlaceholder) {
+		LinkedList<Part> parts = new LinkedList<>();
 		int startIndex = nextStartPrefix(value, 0);
 		if (startIndex == -1) {
-			return null;
+			Part part = (inPlaceholder ? createSimplePlaceholderPart(value) : new TextPart(value));
+			parts.add(part);
+			return parts;
 		}
-		List<Part> parts = new ArrayList<>(4);
 		int position = 0;
 		while (startIndex != -1) {
 			int endIndex = nextValidEndPrefix(value, startIndex);
-			if (endIndex == -1) {  // Not a valid placeholder, consume the prefix and continue
+			if (endIndex == -1) { // Not a valid placeholder, consume the prefix and continue
 				addText(value, position, startIndex + this.prefix.length(), parts);
 				position = startIndex + this.prefix.length();
 				startIndex = nextStartPrefix(value, position);
 			}
-			else if (isEscaped(value, startIndex)) {  // Not a valid index, accumulate and skip the escape character
+			else if (isEscaped(value, startIndex)) { // Not a valid index, accumulate and skip the escape character
 				addText(value, position, startIndex - 1, parts);
 				addText(value, startIndex, startIndex + this.prefix.length(), parts);
 				position = startIndex + this.prefix.length();
 				startIndex = nextStartPrefix(value, position);
 			}
-			else {  // Found valid placeholder, recursive parsing
+			else { // Found valid placeholder, recursive parsing
 				addText(value, position, startIndex, parts);
 				String placeholder = value.substring(startIndex + this.prefix.length(), endIndex);
 				List<Part> placeholderParts = parse(placeholder, true);
-				if (placeholderParts == null) {
-					parts.add(createSimplePlaceholderPart(placeholder));
-				}
-				else {
-					parts.addAll(placeholderParts);
-				}
+				parts.addAll(placeholderParts);
 				startIndex = nextStartPrefix(value, endIndex + this.suffix.length());
 				position = endIndex + this.suffix.length();
 			}
@@ -237,6 +241,29 @@ final class PlaceholderParser {
 		return new ParsedSection(buffer.toString(), null);
 	}
 
+	private static void addText(String value, int start, int end, LinkedList<Part> parts) {
+		if (start > end) {
+			return;
+		}
+		String text = value.substring(start, end);
+		if (!text.isEmpty()) {
+			if (!parts.isEmpty()) {
+				Part current = parts.removeLast();
+				if (current instanceof TextPart textPart) {
+					parts.add(new TextPart(textPart.text() + text));
+				}
+				else {
+					parts.add(current);
+					parts.add(new TextPart(text));
+				}
+			}
+			else {
+				parts.add(new TextPart(text));
+			}
+		}
+	}
+
+
 	private int nextStartPrefix(String value, int index) {
 		return value.indexOf(this.prefix, index);
 	}
@@ -269,46 +296,15 @@ final class PlaceholderParser {
 		return (this.escape != null && index > 0 && value.charAt(index - 1) == this.escape);
 	}
 
-	private static void addText(String value, int start, int end, List<Part> parts) {
-		if (start >= end) {
-			return;
-		}
-		String text = value.substring(start, end);
-		if (!parts.isEmpty() && parts.get(parts.size() - 1) instanceof TextPart textPart) {
-			parts.set(parts.size() - 1, new TextPart(textPart.text() + text));
-		}
-		else {
-			parts.add(new TextPart(text));
-		}
-	}
+	record ParsedSection(String key, @Nullable String fallback) {
 
-
-	/**
-	 * A representation of the parsing of an input string.
-	 * @param text the raw input string
-	 * @param parts the parts that appear in the string, in order
-	 */
-	private record ParsedValue(String text, List<Part> parts) {
-
-		public String resolve(PartResolutionContext resolutionContext) {
-			try {
-				return Part.resolveAll(this.parts, resolutionContext);
-			}
-			catch (PlaceholderResolutionException ex) {
-				throw ex.withValue(this.text);
-			}
-		}
-	}
-
-
-	private record ParsedSection(String key, @Nullable String fallback) {
 	}
 
 
 	/**
 	 * Provide the necessary context to handle and resolve underlying placeholders.
 	 */
-	private static class PartResolutionContext implements PlaceholderResolver {
+	static class PartResolutionContext implements PlaceholderResolver {
 
 		private final String prefix;
 
@@ -323,6 +319,7 @@ final class PlaceholderParser {
 		@Nullable
 		private Set<String> visitedPlaceholders;
 
+
 		PartResolutionContext(PlaceholderResolver resolver, String prefix, String suffix,
 				boolean ignoreUnresolvablePlaceholders, Function<String, List<Part>> parser) {
 			this.prefix = prefix;
@@ -355,7 +352,7 @@ final class PlaceholderParser {
 			return this.prefix + text + this.suffix;
 		}
 
-		public @Nullable List<Part> parse(String text) {
+		public List<Part> parse(String text) {
 			return this.parser.apply(text);
 		}
 
@@ -370,17 +367,17 @@ final class PlaceholderParser {
 		}
 
 		public void removePlaceholder(String placeholder) {
-			if (this.visitedPlaceholders != null) {
-				this.visitedPlaceholders.remove(placeholder);
-			}
+			Assert.state(this.visitedPlaceholders != null, "Visited placeholders must not be null");
+			this.visitedPlaceholders.remove(placeholder);
 		}
+
 	}
 
 
 	/**
 	 * A part is a section of a String containing placeholders to replace.
 	 */
-	private interface Part {
+	interface Part {
 
 		/**
 		 * Resolve this part using the specified {@link PartResolutionContext}.
@@ -411,12 +408,30 @@ final class PlaceholderParser {
 	}
 
 
+	/**
+	 * A representation of the parsing of an input string.
+	 * @param text the raw input string
+	 * @param parts the parts that appear in the string, in order
+	 */
+	record ParsedValue(String text, List<Part> parts) {
+
+		public String resolve(PartResolutionContext resolutionContext) {
+			try {
+				return Part.resolveAll(this.parts, resolutionContext);
+			}
+			catch (PlaceholderResolutionException ex) {
+				throw ex.withValue(this.text);
+			}
+		}
+	}
+
+
 	/**
 	 * A base {@link Part} implementation.
 	 */
-	private abstract static class AbstractPart implements Part {
+	abstract static class AbstractPart implements Part {
 
-		final String text;
+		private final String text;
 
 		protected AbstractPart(String text) {
 			this.text = text;
@@ -439,19 +454,29 @@ final class PlaceholderParser {
 		@Nullable
 		protected String resolveRecursively(PartResolutionContext resolutionContext, String key) {
 			String resolvedValue = resolutionContext.resolvePlaceholder(key);
-			if (resolvedValue == null) {
-				// Not found
-				return null;
+			if (resolvedValue != null) {
+				resolutionContext.flagPlaceholderAsVisited(key);
+				// Let's check if we need to recursively resolve that value
+				List<Part> nestedParts = resolutionContext.parse(resolvedValue);
+				String value = toText(nestedParts);
+				if (!isTextOnly(nestedParts)) {
+					value = new ParsedValue(resolvedValue, nestedParts).resolve(resolutionContext);
+				}
+				resolutionContext.removePlaceholder(key);
+				return value;
 			}
-			// Let's check if we need to recursively resolve that value
-			List<Part> nestedParts = resolutionContext.parse(resolvedValue);
-			if (nestedParts == null) {
-				return resolvedValue;
-			}
-			resolutionContext.flagPlaceholderAsVisited(key);
-			String value = new ParsedValue(resolvedValue, nestedParts).resolve(resolutionContext);
-			resolutionContext.removePlaceholder(key);
-			return value;
+			// Not found
+			return null;
+		}
+
+		private boolean isTextOnly(List<Part> parts) {
+			return parts.stream().allMatch(TextPart.class::isInstance);
+		}
+
+		private String toText(List<Part> parts) {
+			StringBuilder sb = new StringBuilder();
+			parts.forEach(part -> sb.append(part.text()));
+			return sb.toString();
 		}
 	}
 
@@ -459,7 +484,7 @@ final class PlaceholderParser {
 	/**
 	 * A {@link Part} implementation that does not contain a valid placeholder.
 	 */
-	private static class TextPart extends AbstractPart {
+	static class TextPart extends AbstractPart {
 
 		/**
 		 * Create a new instance.
@@ -471,7 +496,7 @@ final class PlaceholderParser {
 
 		@Override
 		public String resolve(PartResolutionContext resolutionContext) {
-			return this.text;
+			return text();
 		}
 	}
 
@@ -480,7 +505,7 @@ final class PlaceholderParser {
 	 * A {@link Part} implementation that represents a single placeholder with
 	 * a hard-coded fallback.
 	 */
-	private static class SimplePlaceholderPart extends AbstractPart {
+	static class SimplePlaceholderPart extends AbstractPart {
 
 		private final String key;
 
@@ -508,13 +533,13 @@ final class PlaceholderParser {
 			else if (this.fallback != null) {
 				return this.fallback;
 			}
-			return resolutionContext.handleUnresolvablePlaceholder(this.key, this.text);
+			return resolutionContext.handleUnresolvablePlaceholder(this.key, text());
 		}
 
 		@Nullable
 		private String resolveRecursively(PartResolutionContext resolutionContext) {
-			if (!this.text.equals(this.key)) {
-				String value = resolveRecursively(resolutionContext, this.text);
+			if (!this.text().equals(this.key)) {
+				String value = resolveRecursively(resolutionContext, this.text());
 				if (value != null) {
 					return value;
 				}
@@ -528,7 +553,7 @@ final class PlaceholderParser {
 	 * A {@link Part} implementation that represents a single placeholder
 	 * containing nested placeholders.
 	 */
-	private static class NestedPlaceholderPart extends AbstractPart {
+	static class NestedPlaceholderPart extends AbstractPart {
 
 		private final List<Part> keyParts;
 
@@ -557,7 +582,7 @@ final class PlaceholderParser {
 			else if (this.defaultParts != null) {
 				return Part.resolveAll(this.defaultParts, resolutionContext);
 			}
-			return resolutionContext.handleUnresolvablePlaceholder(resolvedKey, this.text);
+			return resolutionContext.handleUnresolvablePlaceholder(resolvedKey, text());
 		}
 	}
 

spring-core/src/main/java/org/springframework/util/PropertyPlaceholderHelper.java

@@ -97,7 +97,7 @@ public class PropertyPlaceholderHelper {
 	 * @param properties the {@code Properties} to use for replacement
 	 * @return the supplied value with placeholders replaced inline
 	 */
-	public String replacePlaceholders(String value, Properties properties) {
+	public String replacePlaceholders(String value, final Properties properties) {
 		Assert.notNull(properties, "'properties' must not be null");
 		return replacePlaceholders(value, properties::getProperty);
 	}
@@ -111,10 +111,9 @@ public class PropertyPlaceholderHelper {
 	 */
 	public String replacePlaceholders(String value, PlaceholderResolver placeholderResolver) {
 		Assert.notNull(value, "'value' must not be null");
-		return this.parser.replacePlaceholders(value, placeholderResolver);
+		return parseStringValue(value, placeholderResolver);
 	}
 
-	@Deprecated(since = "6.2.12", forRemoval = true)
 	protected String parseStringValue(String value, PlaceholderResolver placeholderResolver) {
 		return this.parser.replacePlaceholders(value, placeholderResolver);
 	}

spring-core/src/main/java/org/springframework/util/StringUtils.java

@@ -16,6 +16,7 @@
 
 package org.springframework.util;
 
+import java.io.ByteArrayOutputStream;
 import java.nio.charset.Charset;
 import java.util.ArrayDeque;
 import java.util.ArrayList;
@@ -24,7 +25,6 @@ import java.util.Collection;
 import java.util.Collections;
 import java.util.Deque;
 import java.util.Enumeration;
-import java.util.HexFormat;
 import java.util.Iterator;
 import java.util.LinkedHashSet;
 import java.util.List;
@@ -803,60 +803,54 @@ public abstract class StringUtils {
 	}
 
 	/**
-	 * Decode the given encoded URI component value by replacing "<i>{@code %xy}</i>" sequences
-	 * by an hexadecimal representation of the character in the specified charset, letting other
-	 * characters unchanged.
-	 * @param source the encoded {@code String}
-	 * @param charset the character encoding to use to decode the "<i>{@code %xy}</i>" sequences
+	 * Decode the given encoded URI component value. Based on the following rules:
+	 * <ul>
+	 * <li>Alphanumeric characters {@code "a"} through {@code "z"}, {@code "A"} through {@code "Z"},
+	 * and {@code "0"} through {@code "9"} stay the same.</li>
+	 * <li>Special characters {@code "-"}, {@code "_"}, {@code "."}, and {@code "*"} stay the same.</li>
+	 * <li>A sequence "<i>{@code %xy}</i>" is interpreted as a hexadecimal representation of the character.</li>
+	 * <li>For all other characters (including those already decoded), the output is undefined.</li>
+	 * </ul>
+	 * @param source the encoded String
+	 * @param charset the character set
 	 * @return the decoded value
 	 * @throws IllegalArgumentException when the given source contains invalid encoded sequences
 	 * @since 5.0
-	 * @see java.net.URLDecoder#decode(String, String) java.net.URLDecoder#decode for HTML form decoding
+	 * @see java.net.URLDecoder#decode(String, String)
 	 */
 	public static String uriDecode(String source, Charset charset) {
 		int length = source.length();
-		int firstPercentIndex = source.indexOf('%');
-		if (length == 0 || firstPercentIndex < 0) {
+		if (length == 0) {
 			return source;
 		}
+		Assert.notNull(charset, "Charset must not be null");
 
-		StringBuilder output = new StringBuilder(length);
-		output.append(source, 0, firstPercentIndex);
-		byte[] bytes = null;
-		int i = firstPercentIndex;
-		while (i < length) {
-			char ch = source.charAt(i);
+		ByteArrayOutputStream baos = new ByteArrayOutputStream(length);
+		boolean changed = false;
+		for (int i = 0; i < length; i++) {
+			int ch = source.charAt(i);
 			if (ch == '%') {
-				try {
-					if (bytes == null) {
-						bytes = new byte[(length - i) / 3];
+				if (i + 2 < length) {
+					char hex1 = source.charAt(i + 1);
+					char hex2 = source.charAt(i + 2);
+					int u = Character.digit(hex1, 16);
+					int l = Character.digit(hex2, 16);
+					if (u == -1 || l == -1) {
+						throw new IllegalArgumentException("Invalid encoded sequence \"" + source.substring(i) + "\"");
 					}
-
-					int pos = 0;
-					while (i + 2 < length && ch == '%') {
-						bytes[pos++] = (byte) HexFormat.fromHexDigits(source, i + 1, i + 3);
-						i += 3;
-						if (i < length) {
-							ch = source.charAt(i);
-						}
-					}
-
-					if (i < length && ch == '%') {
-						throw new IllegalArgumentException("Incomplete trailing escape (%) pattern");
-					}
-
-					output.append(new String(bytes, 0, pos, charset));
+					baos.write((char) ((u << 4) + l));
+					i += 2;
+					changed = true;
 				}
-				catch (NumberFormatException ex) {
+				else {
 					throw new IllegalArgumentException("Invalid encoded sequence \"" + source.substring(i) + "\"");
 				}
 			}
 			else {
-				output.append(ch);
-				i++;
+				baos.write(ch);
 			}
 		}
-		return output.toString();
+		return (changed ? StreamUtils.copyToString(baos, charset) : source);
 	}
 
 	/**

spring-core/src/main/java/org/springframework/util/unit/DataSize.java

@@ -137,19 +137,14 @@ public final class DataSize implements Comparable<DataSize>, Serializable {
 	}
 
 	/**
-	 * Obtain a {@link DataSize} from a text string such as {@code "5MB"} using
+	 * Obtain a {@link DataSize} from a text string such as {@code 12MB} using
 	 * {@link DataUnit#BYTES} if no unit is specified.
-	 * <h4>Examples</h4>
-	 * <table border="1">
-	 * <tr><th>Text</th><th>Parsed As</th><th>Size in Bytes</th></tr>
-	 * <tr><td>"20"</td><td>20 bytes</td><td>20</td></tr>
-	 * <tr><td>"20B"</td><td>20 bytes</td><td>20</td></tr>
-	 * <tr><td>"12KB"</td><td>12 kilobytes</td><td>12,288</td></tr>
-	 * <tr><td>"5MB"</td><td>5 megabytes</td><td>5,242,880</td></tr>
-	 * </table>
-	 * <p>Note that the terms and units used in the above examples are based on
-	 * <a href="https://en.wikipedia.org/wiki/Binary_prefix">binary prefixes</a>.
-	 * Consult the {@linkplain DataSize class-level Javadoc} for details.
+	 * <p>Examples:
+	 * <pre>
+	 * "12KB" -- parses as "12 kilobytes"
+	 * "5MB"  -- parses as "5 megabytes"
+	 * "20"   -- parses as "20 bytes"
+	 * </pre>
 	 * @param text the text to parse
 	 * @return the parsed {@code DataSize}
 	 * @see #parse(CharSequence, DataUnit)
@@ -159,24 +154,19 @@ public final class DataSize implements Comparable<DataSize>, Serializable {
 	}
 
 	/**
-	 * Obtain a {@link DataSize} from a text string such as {@code "5MB"} using
+	 * Obtain a {@link DataSize} from a text string such as {@code 12MB} using
 	 * the specified default {@link DataUnit} if no unit is specified.
 	 * <p>The string starts with a number followed optionally by a unit matching
 	 * one of the supported {@linkplain DataUnit suffixes}.
 	 * <p>If neither a unit nor a default {@code DataUnit} is specified,
 	 * {@link DataUnit#BYTES} will be inferred.
-	 * <h4>Examples</h4>
-	 * <table border="1">
-	 * <tr><th>Text</th><th>Default Unit</th><th>Parsed As</th><th>Size in Bytes</th></tr>
-	 * <tr><td>"20"</td><td>{@code null}</td><td>20 bytes</td><td>20</td></tr>
-	 * <tr><td>"20"</td><td>{@link DataUnit#KILOBYTES KILOBYTES}</td><td>20 kilobytes</td><td>20,480</td></tr>
-	 * <tr><td>"20B"</td><td>N/A</td><td>20 bytes</td><td>20</td></tr>
-	 * <tr><td>"12KB"</td><td>N/A</td><td>12 kilobytes</td><td>12,288</td></tr>
-	 * <tr><td>"5MB"</td><td>N/A</td><td>5 megabytes</td><td>5,242,880</td></tr>
-	 * </table>
-	 * <p>Note that the terms and units used in the above examples are based on
-	 * <a href="https://en.wikipedia.org/wiki/Binary_prefix">binary prefixes</a>.
-	 * Consult the {@linkplain DataSize class-level Javadoc} for details.
+	 * <p>Examples:
+	 * <pre>
+	 * "12KB" -- parses as "12 kilobytes"
+	 * "5MB"  -- parses as "5 megabytes"
+	 * "20"   -- parses as "20 kilobytes" (where the {@code defaultUnit} is {@link DataUnit#KILOBYTES})
+	 * "20"   -- parses as "20 bytes" (if the {@code defaultUnit} is {@code null})
+	 * </pre>
 	 * @param text the text to parse
 	 * @param defaultUnit the default {@code DataUnit} to use
 	 * @return the parsed {@code DataSize}

spring-core/src/test/java/org/springframework/core/annotation/AnnotatedMethodTests.java

@@ -1,133 +0,0 @@
-/*
- * Copyright 2002-present 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.
- * You may obtain a copy of the License at
- *
- *      https://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-
-package org.springframework.core.annotation;
-
-import java.lang.annotation.Retention;
-import java.lang.annotation.RetentionPolicy;
-import java.lang.reflect.Method;
-import java.lang.reflect.Modifier;
-
-import org.junit.jupiter.api.Test;
-
-import org.springframework.core.MethodParameter;
-import org.springframework.util.ReflectionUtils;
-
-import static java.util.Arrays.stream;
-import static java.util.stream.Collectors.joining;
-import static org.assertj.core.api.Assertions.assertThat;
-
-/**
- * Tests for {@link AnnotatedMethod}.
- *
- * @author Sam Brannen
- * @since 6.2.11
- */
-class AnnotatedMethodTests {
-
-	@Test
-	void shouldFindAnnotationOnMethodInGenericAbstractSuperclass() {
-		Method processTwo = getMethod("processTwo", String.class);
-
-		AnnotatedMethod annotatedMethod = new AnnotatedMethod(processTwo);
-
-		assertThat(annotatedMethod.hasMethodAnnotation(Handler.class)).isTrue();
-	}
-
-	@Test
-	void shouldFindAnnotationOnMethodInGenericInterface() {
-		Method processOneAndTwo = getMethod("processOneAndTwo", Long.class, Object.class);
-
-		AnnotatedMethod annotatedMethod = new AnnotatedMethod(processOneAndTwo);
-
-		assertThat(annotatedMethod.hasMethodAnnotation(Handler.class)).isTrue();
-	}
-
-	@Test
-	void shouldFindAnnotationOnMethodParameterInGenericAbstractSuperclass() {
-		// Prerequisites for gh-35349
-		Method abstractMethod = ReflectionUtils.findMethod(GenericAbstractSuperclass.class, "processTwo", Object.class);
-		assertThat(abstractMethod).isNotNull();
-		assertThat(Modifier.isAbstract(abstractMethod.getModifiers())).as("abstract").isTrue();
-		assertThat(Modifier.isPublic(abstractMethod.getModifiers())).as("public").isFalse();
-
-		Method processTwo = getMethod("processTwo", String.class);
-
-		AnnotatedMethod annotatedMethod = new AnnotatedMethod(processTwo);
-		MethodParameter[] methodParameters = annotatedMethod.getMethodParameters();
-
-		assertThat(methodParameters).hasSize(1);
-		assertThat(methodParameters[0].hasParameterAnnotation(Param.class)).isTrue();
-	}
-
-	@Test
-	void shouldFindAnnotationOnMethodParameterInGenericInterface() {
-		Method processOneAndTwo = getMethod("processOneAndTwo", Long.class, Object.class);
-
-		AnnotatedMethod annotatedMethod = new AnnotatedMethod(processOneAndTwo);
-		MethodParameter[] methodParameters = annotatedMethod.getMethodParameters();
-
-		assertThat(methodParameters).hasSize(2);
-		assertThat(methodParameters[0].hasParameterAnnotation(Param.class)).isFalse();
-		assertThat(methodParameters[1].hasParameterAnnotation(Param.class)).isTrue();
-	}
-
-
-	private static Method getMethod(String name, Class<?>...parameterTypes) {
-		Class<?> clazz = GenericInterfaceImpl.class;
-		Method method = ReflectionUtils.findMethod(clazz, name, parameterTypes);
-		if (method == null) {
-			String parameterNames = stream(parameterTypes).map(Class::getName).collect(joining(", "));
-			throw new IllegalStateException("Expected method not found: %s#%s(%s)"
-					.formatted(clazz.getSimpleName(), name, parameterNames));
-		}
-		return method;
-	}
-
-
-	@Retention(RetentionPolicy.RUNTIME)
-	@interface Handler {
-	}
-
-	@Retention(RetentionPolicy.RUNTIME)
-	@interface Param {
-	}
-
-	interface GenericInterface<A, B> {
-
-		@Handler
-		void processOneAndTwo(A value1, @Param B value2);
-	}
-
-	abstract static class GenericAbstractSuperclass<C> implements GenericInterface<Long, C> {
-
-		@Override
-		public void processOneAndTwo(Long value1, C value2) {
-		}
-
-		@Handler
-		// Intentionally NOT public
-		abstract void processTwo(@Param C value);
-	}
-
-	static class GenericInterfaceImpl extends GenericAbstractSuperclass<String> {
-
-		@Override
-		void processTwo(String value) {
-		}
-	}
-
-}

spring-core/src/test/java/org/springframework/core/annotation/MergedAnnotationsTests.java

@@ -945,15 +945,6 @@ class MergedAnnotationsTests {
 				Order.class).getDistance()).isEqualTo(0);
 	}
 
-	@Test
-	void getFromMethodWithUnresolvedGenericsInGenericTypeHierarchy() {
-		// The following method is GenericAbstractSuperclass.processOneAndTwo(java.lang.Long, C),
-		// where 'C' is an unresolved generic, for which ResolvableType.resolve() returns null.
-		Method method = ClassUtils.getMethod(GenericInterfaceImpl.class, "processOneAndTwo", Long.class, Object.class);
-		assertThat(MergedAnnotations.from(method, SearchStrategy.TYPE_HIERARCHY)
-				.get(Transactional.class).isDirectlyPresent()).isTrue();
-	}
-
 	@Test
 	void getFromMethodWithInterfaceOnSuper() throws Exception {
 		Method method = SubOfImplementsInterfaceWithAnnotatedMethod.class.getMethod("foo");
@@ -3041,26 +3032,6 @@ class MergedAnnotationsTests {
 		}
 	}
 
-	interface GenericInterface<A, B> {
-
-		@Transactional
-		void processOneAndTwo(A value1, B value2);
-	}
-
-	abstract static class GenericAbstractSuperclass<C> implements GenericInterface<Long, C> {
-
-		@Override
-		public void processOneAndTwo(Long value1, C value2) {
-		}
-	}
-
-	static class GenericInterfaceImpl extends GenericAbstractSuperclass<String> {
-		// The compiler does not require us to declare a concrete
-		// processOneAndTwo(Long, String) method, and we intentionally
-		// do not declare one here.
-	}
-
-
 	@Retention(RetentionPolicy.RUNTIME)
 	@Inherited
 	@interface MyRepeatableContainer {

spring-core/src/test/java/org/springframework/core/io/buffer/DataBufferTests.java

@@ -21,9 +21,7 @@ import java.io.InputStream;
 import java.io.OutputStream;
 import java.nio.ByteBuffer;
 import java.nio.charset.StandardCharsets;
-import java.util.ArrayList;
 import java.util.Arrays;
-import java.util.List;
 
 import org.springframework.core.testfixture.io.buffer.AbstractDataBufferAllocatingTests;
 
@@ -1047,35 +1045,4 @@ class DataBufferTests extends AbstractDataBufferAllocatingTests {
 		release(buffer);
 	}
 
-	@ParameterizedDataBufferAllocatingTest
-	void forEachByteProcessAll(DataBufferFactory bufferFactory) {
-		super.bufferFactory = bufferFactory;
-
-		List<Byte> result = new ArrayList<>();
-		DataBuffer buffer = byteBuffer(new byte[]{'a', 'b', 'c', 'd'});
-		int index = buffer.forEachByte(0, 4, b -> {
-			result.add(b);
-			return true;
-		});
-		assertThat(index).isEqualTo(-1);
-		assertThat(result).containsExactly((byte) 'a', (byte) 'b', (byte) 'c', (byte) 'd');
-		release(buffer);
-	}
-
-
-	@ParameterizedDataBufferAllocatingTest
-	void forEachByteProcessSome(DataBufferFactory bufferFactory) {
-		super.bufferFactory = bufferFactory;
-
-		List<Byte> result = new ArrayList<>();
-		DataBuffer buffer = byteBuffer(new byte[]{'a', 'b', 'c', 'd'});
-		int index = buffer.forEachByte(0, 4, b -> {
-			result.add(b);
-			return (b != 'c');
-		});
-		assertThat(index).isEqualTo(2);
-		assertThat(result).containsExactly((byte) 'a', (byte) 'b', (byte) 'c');
-		release(buffer);
-	}
-
 }

spring-core/src/test/java/org/springframework/core/io/buffer/JettyDataBufferTests.java

@@ -18,34 +18,33 @@ package org.springframework.core.io.buffer;
 
 import java.nio.ByteBuffer;
 
-import org.eclipse.jetty.io.ArrayByteBufferPool;
 import org.eclipse.jetty.io.Content;
-import org.eclipse.jetty.io.RetainableByteBuffer;
-import org.junit.jupiter.api.AfterEach;
 import org.junit.jupiter.api.Test;
 
 import static org.assertj.core.api.Assertions.assertThat;
 import static org.assertj.core.api.Assertions.assertThatIllegalStateException;
+import static org.mockito.BDDMockito.given;
+import static org.mockito.BDDMockito.then;
+import static org.mockito.Mockito.mock;
+import static org.mockito.Mockito.times;
 
 /**
- * Tests for {@link JettyDataBuffer}
  * @author Arjen Poutsma
- * @author Brian Clozel
  */
 public class JettyDataBufferTests {
 
 	private final JettyDataBufferFactory dataBufferFactory = new JettyDataBufferFactory();
 
-	private ArrayByteBufferPool.Tracking byteBufferPool = new ArrayByteBufferPool.Tracking();
-
 	@Test
 	void releaseRetainChunk() {
-		RetainableByteBuffer retainableBuffer = byteBufferPool.acquire(3, false);
-		ByteBuffer buffer = retainableBuffer.getByteBuffer();
-		buffer.position(0).limit(1);
-		Content.Chunk chunk = Content.Chunk.asChunk(buffer, false, retainableBuffer);
+		ByteBuffer buffer = ByteBuffer.allocate(3);
+		Content.Chunk mockChunk = mock();
+		given(mockChunk.getByteBuffer()).willReturn(buffer);
+		given(mockChunk.release()).willReturn(false, false, true);
 
-		JettyDataBuffer dataBuffer = this.dataBufferFactory.wrap(chunk);
+
+
+		JettyDataBuffer dataBuffer = this.dataBufferFactory.wrap(mockChunk);
 		dataBuffer.retain();
 		dataBuffer.retain();
 		assertThat(dataBuffer.release()).isFalse();
@@ -53,12 +52,8 @@ public class JettyDataBufferTests {
 		assertThat(dataBuffer.release()).isTrue();
 
 		assertThatIllegalStateException().isThrownBy(dataBuffer::release);
-		assertThat(retainableBuffer.isRetained()).isFalse();
-		assertThat(byteBufferPool.getLeaks()).isEmpty();
-	}
 
-	@AfterEach
-	public void tearDown() throws Exception {
-		this.byteBufferPool.clear();
+		then(mockChunk).should(times(3)).retain();
+		then(mockChunk).should(times(3)).release();
 	}
 }

spring-core/src/test/java/org/springframework/util/ClassUtilsTests.java

@@ -27,7 +27,6 @@ import java.lang.reflect.Member;
 import java.lang.reflect.Method;
 import java.lang.reflect.Modifier;
 import java.lang.reflect.Proxy;
-import java.net.URLConnection;
 import java.time.ZoneId;
 import java.util.ArrayList;
 import java.util.Arrays;
@@ -688,13 +687,13 @@ class ClassUtilsTests {
 		}
 
 		@Test
-		void publicMethodInPublicClass() throws Exception {
+		void publicMethodInObjectClass() throws Exception {
 			Class<?> originalType = String.class;
-			Method originalMethod = originalType.getDeclaredMethod("toString");
+			Method originalMethod = originalType.getDeclaredMethod("hashCode");
 
 			Method publiclyAccessibleMethod = ClassUtils.getPubliclyAccessibleMethodIfPossible(originalMethod, null);
-			assertThat(publiclyAccessibleMethod.getDeclaringClass()).isEqualTo(originalType);
-			assertThat(publiclyAccessibleMethod).isSameAs(originalMethod);
+			assertThat(publiclyAccessibleMethod.getDeclaringClass()).isEqualTo(Object.class);
+			assertThat(publiclyAccessibleMethod.getName()).isEqualTo("hashCode");
 			assertPubliclyAccessible(publiclyAccessibleMethod);
 		}
 
@@ -704,20 +703,9 @@ class ClassUtilsTests {
 			Method originalMethod = originalType.getDeclaredMethod("size");
 
 			Method publiclyAccessibleMethod = ClassUtils.getPubliclyAccessibleMethodIfPossible(originalMethod, null);
-			// Should not find the interface method in List.
-			assertThat(publiclyAccessibleMethod.getDeclaringClass()).isEqualTo(originalType);
-			assertThat(publiclyAccessibleMethod).isSameAs(originalMethod);
-			assertPubliclyAccessible(publiclyAccessibleMethod);
-		}
-
-		@Test
-		void publicMethodInNonExportedClass() throws Exception {
-			Class<?> originalType = getClass().getClassLoader().loadClass("sun.net.www.protocol.http.HttpURLConnection");
-			Method originalMethod = originalType.getDeclaredMethod("getOutputStream");
-
-			Method publiclyAccessibleMethod = ClassUtils.getPubliclyAccessibleMethodIfPossible(originalMethod, null);
-			assertThat(publiclyAccessibleMethod.getDeclaringClass()).isEqualTo(URLConnection.class);
-			assertThat(publiclyAccessibleMethod.getName()).isSameAs(originalMethod.getName());
+			// Should find the interface method in List.
+			assertThat(publiclyAccessibleMethod.getDeclaringClass()).isEqualTo(List.class);
+			assertThat(publiclyAccessibleMethod.getName()).isEqualTo("size");
 			assertPubliclyAccessible(publiclyAccessibleMethod);
 		}
 

spring-core/src/test/java/org/springframework/util/FileSystemUtilsTests.java

@@ -17,29 +17,20 @@
 package org.springframework.util;
 
 import java.io.File;
-import java.net.URI;
-import java.nio.file.FileSystem;
-import java.nio.file.FileSystems;
-import java.nio.file.Path;
-import java.util.Map;
 
+import org.junit.jupiter.api.AfterEach;
 import org.junit.jupiter.api.Test;
-import org.junit.jupiter.api.io.TempDir;
 
 import static org.assertj.core.api.Assertions.assertThat;
 
 /**
- * Tests for {@link FileSystemUtils}.
- *
  * @author Rob Harrop
- * @author Sam Brannen
- * @author Juergen Hoeller
  */
 class FileSystemUtilsTests {
 
 	@Test
-	void deleteRecursively(@TempDir File tempDir) throws Exception {
-		File root = new File(tempDir, "root");
+	void deleteRecursively() throws Exception {
+		File root = new File("./tmp/root");
 		File child = new File(root, "child");
 		File grandchild = new File(child, "grandchild");
 
@@ -62,8 +53,8 @@ class FileSystemUtilsTests {
 	}
 
 	@Test
-	void copyRecursively(@TempDir File tempDir) throws Exception {
-		File src = new File(tempDir, "src");
+	void copyRecursively() throws Exception {
+		File src = new File("./tmp/src");
 		File child = new File(src, "child");
 		File grandchild = new File(child, "grandchild");
 
@@ -77,29 +68,27 @@ class FileSystemUtilsTests {
 		assertThat(grandchild).exists();
 		assertThat(bar).exists();
 
-		File dest = new File(tempDir, "/dest");
+		File dest = new File("./dest");
 		FileSystemUtils.copyRecursively(src, dest);
 
 		assertThat(dest).exists();
-		assertThat(new File(dest, "child")).exists();
-		assertThat(new File(dest, "child/bar.txt")).exists();
+		assertThat(new File(dest, child.getName())).exists();
 
-		String destPath = dest.toString().replace('\\', '/');
-		if (!destPath.startsWith("/")) {
-			destPath = "/" + destPath;
-		}
-		URI uri = URI.create("jar:file:" + destPath + "/archive.zip");
-		Map<String, String> env = Map.of("create", "true");
-		FileSystem zipfs = FileSystems.newFileSystem(uri, env);
-		Path ziproot = zipfs.getPath("/");
-		FileSystemUtils.copyRecursively(src.toPath(), ziproot);
-
-		assertThat(zipfs.getPath("/child")).exists();
-		assertThat(zipfs.getPath("/child/bar.txt")).exists();
-
-		zipfs.close();
 		FileSystemUtils.deleteRecursively(src);
 		assertThat(src).doesNotExist();
 	}
 
+
+	@AfterEach
+	void tearDown() {
+		File tmp = new File("./tmp");
+		if (tmp.exists()) {
+			FileSystemUtils.deleteRecursively(tmp);
+		}
+		File dest = new File("./dest");
+		if (dest.exists()) {
+			FileSystemUtils.deleteRecursively(dest);
+		}
+	}
+
 }

spring-core/src/test/java/org/springframework/util/PlaceholderParserTests.java

@@ -26,6 +26,8 @@ import org.junit.jupiter.params.provider.Arguments;
 import org.junit.jupiter.params.provider.MethodSource;
 import org.mockito.InOrder;
 
+import org.springframework.util.PlaceholderParser.ParsedValue;
+import org.springframework.util.PlaceholderParser.TextPart;
 import org.springframework.util.PropertyPlaceholderHelper.PlaceholderResolver;
 
 import static org.assertj.core.api.Assertions.assertThat;
@@ -41,11 +43,10 @@ import static org.mockito.Mockito.verifyNoMoreInteractions;
  *
  * @author Stephane Nicoll
  * @author Sam Brannen
- * @author Juergen Hoeller
  */
 class PlaceholderParserTests {
 
-	@Nested  // Tests with only the basic placeholder feature enabled
+	@Nested // Tests with only the basic placeholder feature enabled
 	class OnlyPlaceholderTests {
 
 		private final PlaceholderParser parser = new PlaceholderParser("${", "}", null, null, true);
@@ -81,7 +82,7 @@ class PlaceholderParserTests {
 			Map<String, String> properties = Map.of(
 					"p1", "v1",
 					"p2", "v2",
-					"p3", "${p1}:${p2}",          // nested placeholders
+					"p3", "${p1}:${p2}",              // nested placeholders
 					"p4", "${p3}",                    // deeply nested placeholders
 					"p5", "${p1}:${p2}:${bogus}");    // unresolvable placeholder
 			assertThat(this.parser.replacePlaceholders(text, properties::get)).isEqualTo(expected);
@@ -153,13 +154,14 @@ class PlaceholderParserTests {
 		@Test
 		void textWithInvalidPlaceholderSyntaxIsMerged() {
 			String text = "test${of${with${and${";
-			assertThat(this.parser.replacePlaceholders(text,
-					placeholder -> {throw new UnsupportedOperationException();})).isEqualTo(text);
+			ParsedValue parsedValue = this.parser.parse(text);
+			assertThat(parsedValue.parts()).singleElement().isInstanceOfSatisfying(TextPart.class,
+					textPart -> assertThat(textPart.text()).isEqualTo(text));
 		}
+
 	}
 
-
-	@Nested  // Tests with the use of a separator
+	@Nested // Tests with the use of a separator
 	class DefaultValueTests {
 
 		private final PlaceholderParser parser = new PlaceholderParser("${", "}", ":", null, true);
@@ -193,7 +195,7 @@ class PlaceholderParserTests {
 			Map<String, String> properties = Map.of(
 					"p1", "v1",
 					"p2", "v2",
-					"p3", "${p1}:${p2}",           // nested placeholders
+					"p3", "${p1}:${p2}",               // nested placeholders
 					"p4", "${p3}",                     // deeply nested placeholders
 					"p5", "${p1}:${p2}:${bogus}",      // unresolvable placeholder
 					"p6", "${p1}:${p2}:${bogus:def}"); // unresolvable w/ default
@@ -257,8 +259,8 @@ class PlaceholderParserTests {
 			assertThat(this.parser.replacePlaceholders("${invalid:${firstName}}", resolver)).isEqualTo("John");
 			verifyPlaceholderResolutions(resolver, "invalid", "firstName");
 		}
-	}
 
+	}
 
 	/**
 	 * Tests that use the escape character.
@@ -339,8 +341,8 @@ class PlaceholderParserTests {
 					Arguments.of("${service/host/${app.environment}/name:\\value}", "https://example.com/qa/name"),
 					Arguments.of("${service/host/${name\\:value}/}", "${service/host/${name:value}/}"));
 		}
-	}
 
+	}
 
 	@Nested
 	class ExceptionTests {
@@ -376,6 +378,7 @@ class PlaceholderParserTests {
 					.withMessage("Could not resolve placeholder 'bogus' in value \"${p1}:${p2}:${bogus}\" <-- \"${p3}\"")
 					.withNoCause();
 		}
+
 	}
 
 

spring-expression/src/main/java/org/springframework/expression/spel/support/StandardEvaluationContext.java

@@ -129,72 +129,34 @@ public class StandardEvaluationContext implements EvaluationContext {
 	}
 
 
-	/**
-	 * Specify the default root context object (including a type descriptor)
-	 * against which unqualified properties, methods, etc. should be resolved.
-	 * @param rootObject the root object to use
-	 * @param typeDescriptor a corresponding type descriptor
-	 */
 	public void setRootObject(@Nullable Object rootObject, TypeDescriptor typeDescriptor) {
 		this.rootObject = new TypedValue(rootObject, typeDescriptor);
 	}
 
-	/**
-	 * Specify the default root context object against which unqualified
-	 * properties, methods, etc. should be resolved.
-	 * @param rootObject the root object to use
-	 */
 	public void setRootObject(@Nullable Object rootObject) {
 		this.rootObject = (rootObject != null ? new TypedValue(rootObject) : TypedValue.NULL);
 	}
 
-	/**
-	 * Return the configured default root context object against which unqualified
-	 * properties, methods, etc. should be resolved (can be {@link TypedValue#NULL}).
-	 */
 	@Override
 	public TypedValue getRootObject() {
 		return this.rootObject;
 	}
 
-	/**
-	 * Set the list of property accessors to use in this evaluation context.
-	 * <p>Replaces any previously configured property accessors.
-	 */
 	public void setPropertyAccessors(List<PropertyAccessor> propertyAccessors) {
 		this.propertyAccessors = propertyAccessors;
 	}
 
-	/**
-	 * Get the list of property accessors configured in this evaluation context.
-	 */
 	@Override
 	public List<PropertyAccessor> getPropertyAccessors() {
 		return initPropertyAccessors();
 	}
 
-	/**
-	 * Add the supplied property accessor to this evaluation context.
-	 * @param propertyAccessor the property accessor to add
-	 * @see #getPropertyAccessors()
-	 * @see #setPropertyAccessors(List)
-	 * @see #removePropertyAccessor(PropertyAccessor)
-	 */
-	public void addPropertyAccessor(PropertyAccessor propertyAccessor) {
-		addBeforeDefault(initPropertyAccessors(), propertyAccessor);
+	public void addPropertyAccessor(PropertyAccessor accessor) {
+		addBeforeDefault(initPropertyAccessors(), accessor);
 	}
 
-	/**
-	 * Remove the supplied property accessor from this evaluation context.
-	 * @param propertyAccessor the property accessor to remove
-	 * @return {@code true} if the property accessor was removed, {@code false}
-	 * if the property accessor was not configured in this evaluation context
-	 * @see #getPropertyAccessors()
-	 * @see #setPropertyAccessors(List)
-	 * @see #addPropertyAccessor(PropertyAccessor)
-	 */
-	public boolean removePropertyAccessor(PropertyAccessor propertyAccessor) {
-		return initPropertyAccessors().remove(propertyAccessor);
+	public boolean removePropertyAccessor(PropertyAccessor accessor) {
+		return initPropertyAccessors().remove(accessor);
 	}
 
 	/**
@@ -236,8 +198,8 @@ public class StandardEvaluationContext implements EvaluationContext {
 	/**
 	 * Remove the supplied index accessor from this evaluation context.
 	 * @param indexAccessor the index accessor to remove
-	 * @return {@code true} if the index accessor was removed, {@code false}
-	 * if the index accessor was not configured in this evaluation context
+	 * @return {@code true} if the index accessor was removed, {@code false} if
+	 * the index accessor was not configured in this evaluation context
 	 * @since 6.2
 	 * @see #getIndexAccessors()
 	 * @see #setIndexAccessors(List)
@@ -247,96 +209,44 @@ public class StandardEvaluationContext implements EvaluationContext {
 		return initIndexAccessors().remove(indexAccessor);
 	}
 
-	/**
-	 * Set the list of constructor resolvers to use in this evaluation context.
-	 * <p>Replaces any previously configured constructor resolvers.
-	 */
 	public void setConstructorResolvers(List<ConstructorResolver> constructorResolvers) {
 		this.constructorResolvers = constructorResolvers;
 	}
 
-	/**
-	 * Get the list of constructor resolvers to use in this evaluation context.
-	 */
 	@Override
 	public List<ConstructorResolver> getConstructorResolvers() {
 		return initConstructorResolvers();
 	}
 
-	/**
-	 * Add the supplied constructor resolver to this evaluation context.
-	 * @param constructorResolver the constructor resolver to add
-	 * @see #getConstructorResolvers()
-	 * @see #setConstructorResolvers(List)
-	 * @see #removeConstructorResolver(ConstructorResolver)
-	 */
-	public void addConstructorResolver(ConstructorResolver constructorResolver) {
-		addBeforeDefault(initConstructorResolvers(), constructorResolver);
+	public void addConstructorResolver(ConstructorResolver resolver) {
+		addBeforeDefault(initConstructorResolvers(), resolver);
 	}
 
-	/**
-	 * Remove the supplied constructor resolver from this evaluation context.
-	 * @param constructorResolver the constructor resolver to remove
-	 * @return {@code true} if the constructor resolver was removed, {@code false}
-	 * if the constructor resolver was not configured in this evaluation context
-	 * @see #getConstructorResolvers()
-	 * @see #setConstructorResolvers(List)
-	 * @see #addConstructorResolver(ConstructorResolver)
-	 */
-	public boolean removeConstructorResolver(ConstructorResolver constructorResolver) {
-		return initConstructorResolvers().remove(constructorResolver);
+	public boolean removeConstructorResolver(ConstructorResolver resolver) {
+		return initConstructorResolvers().remove(resolver);
 	}
 
-	/**
-	 * Set the list of method resolvers to use in this evaluation context.
-	 * <p>Replaces any previously configured method resolvers.
-	 */
 	public void setMethodResolvers(List<MethodResolver> methodResolvers) {
 		this.methodResolvers = methodResolvers;
 	}
 
-	/**
-	 * Get the list of method resolvers to use in this evaluation context.
-	 */
 	@Override
 	public List<MethodResolver> getMethodResolvers() {
 		return initMethodResolvers();
 	}
 
-	/**
-	 * Add the supplied method resolver to this evaluation context.
-	 * @param methodResolver the method resolver to add
-	 * @see #getMethodResolvers()
-	 * @see #setMethodResolvers(List)
-	 * @see #removeMethodResolver(MethodResolver)
-	 */
-	public void addMethodResolver(MethodResolver methodResolver) {
-		addBeforeDefault(initMethodResolvers(), methodResolver);
+	public void addMethodResolver(MethodResolver resolver) {
+		addBeforeDefault(initMethodResolvers(), resolver);
 	}
 
-	/**
-	 * Remove the supplied method resolver from this evaluation context.
-	 * @param methodResolver the method resolver to remove
-	 * @return {@code true} if the method resolver was removed, {@code false}
-	 * if the method resolver was not configured in this evaluation context
-	 * @see #getMethodResolvers()
-	 * @see #setMethodResolvers(List)
-	 * @see #addMethodResolver(MethodResolver)
-	 */
 	public boolean removeMethodResolver(MethodResolver methodResolver) {
 		return initMethodResolvers().remove(methodResolver);
 	}
 
-	/**
-	 * Set the {@link BeanResolver} to use for looking up beans, if any.
-	 */
-	public void setBeanResolver(@Nullable BeanResolver beanResolver) {
+	public void setBeanResolver(BeanResolver beanResolver) {
 		this.beanResolver = beanResolver;
 	}
 
-	/**
-	 * Get the configured {@link BeanResolver} for looking up beans, if any.
-	 */
 	@Override
 	@Nullable
 	public BeanResolver getBeanResolver() {
@@ -374,17 +284,11 @@ public class StandardEvaluationContext implements EvaluationContext {
 		return this.typeLocator;
 	}
 
-	/**
-	 * Set the {@link TypeConverter} for value conversion.
-	 */
 	public void setTypeConverter(TypeConverter typeConverter) {
 		Assert.notNull(typeConverter, "TypeConverter must not be null");
 		this.typeConverter = typeConverter;
 	}
 
-	/**
-	 * Get the configured {@link TypeConverter} for value conversion.
-	 */
 	@Override
 	public TypeConverter getTypeConverter() {
 		if (this.typeConverter == null) {
@@ -393,33 +297,21 @@ public class StandardEvaluationContext implements EvaluationContext {
 		return this.typeConverter;
 	}
 
-	/**
-	 * Set the {@link TypeComparator} for comparing pairs of objects.
-	 */
 	public void setTypeComparator(TypeComparator typeComparator) {
 		Assert.notNull(typeComparator, "TypeComparator must not be null");
 		this.typeComparator = typeComparator;
 	}
 
-	/**
-	 * Get the configured {@link TypeComparator} for comparing pairs of objects.
-	 */
 	@Override
 	public TypeComparator getTypeComparator() {
 		return this.typeComparator;
 	}
 
-	/**
-	 * Set the {@link OperatorOverloader} for mathematical operations.
-	 */
 	public void setOperatorOverloader(OperatorOverloader operatorOverloader) {
 		Assert.notNull(operatorOverloader, "OperatorOverloader must not be null");
 		this.operatorOverloader = operatorOverloader;
 	}
 
-	/**
-	 * Get the configured {@link OperatorOverloader} for mathematical operations.
-	 */
 	@Override
 	public OperatorOverloader getOperatorOverloader() {
 		return this.operatorOverloader;

spring-jdbc/src/main/java/org/springframework/jdbc/core/metadata/GenericTableMetaDataProvider.java

@@ -21,6 +21,7 @@ import java.sql.ResultSet;
 import java.sql.SQLException;
 import java.sql.Types;
 import java.util.ArrayList;
+import java.util.Arrays;
 import java.util.HashMap;
 import java.util.List;
 import java.util.Locale;
@@ -47,6 +48,11 @@ public class GenericTableMetaDataProvider implements TableMetaDataProvider {
 	/** Logger available to subclasses. */
 	protected static final Log logger = LogFactory.getLog(TableMetaDataProvider.class);
 
+	/** Database products we know not supporting the use of a String[] for generated keys. */
+	private static final List<String> productsNotSupportingGeneratedKeysColumnNameArray =
+			Arrays.asList("Apache Derby", "HSQL Database Engine");
+
+
 	/** The name of the user currently connected. */
 	@Nullable
 	private final String userName;
@@ -89,14 +95,45 @@ public class GenericTableMetaDataProvider implements TableMetaDataProvider {
 	@Override
 	public void initializeWithMetaData(DatabaseMetaData databaseMetaData) throws SQLException {
 		try {
-			setGetGeneratedKeysSupported(databaseMetaData.supportsGetGeneratedKeys());
-			setGeneratedKeysColumnNameArraySupported(isGetGeneratedKeysSupported());
+			if (databaseMetaData.supportsGetGeneratedKeys()) {
+				logger.debug("GetGeneratedKeys is supported");
+				setGetGeneratedKeysSupported(true);
+			}
+			else {
+				logger.debug("GetGeneratedKeys is not supported");
+				setGetGeneratedKeysSupported(false);
+			}
 		}
 		catch (SQLException ex) {
 			if (logger.isWarnEnabled()) {
 				logger.warn("Error retrieving 'DatabaseMetaData.supportsGetGeneratedKeys': " + ex.getMessage());
 			}
 		}
+		try {
+			String databaseProductName = databaseMetaData.getDatabaseProductName();
+			if (productsNotSupportingGeneratedKeysColumnNameArray.contains(databaseProductName)) {
+				if (logger.isDebugEnabled()) {
+					logger.debug("GeneratedKeysColumnNameArray is not supported for " + databaseProductName);
+				}
+				setGeneratedKeysColumnNameArraySupported(false);
+			}
+			else {
+				if (isGetGeneratedKeysSupported()) {
+					if (logger.isDebugEnabled()) {
+						logger.debug("GeneratedKeysColumnNameArray is supported for " + databaseProductName);
+					}
+					setGeneratedKeysColumnNameArraySupported(true);
+				}
+				else {
+					setGeneratedKeysColumnNameArraySupported(false);
+				}
+			}
+		}
+		catch (SQLException ex) {
+			if (logger.isWarnEnabled()) {
+				logger.warn("Error retrieving 'DatabaseMetaData.getDatabaseProductName': " + ex.getMessage());
+			}
+		}
 
 		try {
 			this.databaseVersion = databaseMetaData.getDatabaseProductVersion();
@@ -188,23 +225,19 @@ public class GenericTableMetaDataProvider implements TableMetaDataProvider {
 		}
 	}
 
-	/**
-	 * This implementation delegates to {@link #catalogNameToUse}.
-	 */
 	@Override
 	@Nullable
 	public String metaDataCatalogNameToUse(@Nullable String catalogName) {
 		return catalogNameToUse(catalogName);
 	}
 
-	/**
-	 * This implementation delegates to {@link #schemaNameToUse}.
-	 * @see #getDefaultSchema()
-	 */
 	@Override
 	@Nullable
 	public String metaDataSchemaNameToUse(@Nullable String schemaName) {
-		return schemaNameToUse(schemaName != null ? schemaName : getDefaultSchema());
+		if (schemaName == null) {
+			return schemaNameToUse(getDefaultSchema());
+		}
+		return schemaNameToUse(schemaName);
 	}
 
 	/**
@@ -368,7 +401,7 @@ public class GenericTableMetaDataProvider implements TableMetaDataProvider {
 		try {
 			tableColumns = databaseMetaData.getColumns(
 					metaDataCatalogName, metaDataSchemaName, metaDataTableName, null);
-			while (tableColumns != null && tableColumns.next()) {
+			while (tableColumns.next()) {
 				String columnName = tableColumns.getString("COLUMN_NAME");
 				int dataType = tableColumns.getInt("DATA_TYPE");
 				if (dataType == Types.DECIMAL) {

spring-jdbc/src/main/java/org/springframework/jdbc/core/metadata/MySQLTableMetaDataProvider.java

@@ -1,36 +0,0 @@
-/*
- * Copyright 2002-present 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.
- * You may obtain a copy of the License at
- *
- *      https://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-
-package org.springframework.jdbc.core.metadata;
-
-import java.sql.DatabaseMetaData;
-import java.sql.SQLException;
-
-/**
- * The MySQL/MariaDB specific implementation of {@link TableMetaDataProvider}.
- * Sets {@link #setGeneratedKeysColumnNameArraySupported} to {@code false}.
- *
- * @author Juergen Hoeller
- * @since 6.2.12
- */
-public class MySQLTableMetaDataProvider extends GenericTableMetaDataProvider {
-
-	public MySQLTableMetaDataProvider(DatabaseMetaData databaseMetaData) throws SQLException {
-		super(databaseMetaData);
-		setGeneratedKeysColumnNameArraySupported(false);
-	}
-
-}