spring-projects / spring-projects/spring-framework

Consider standardizing default thread name prefixes for MVC async executors

Open
#36,846 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

status: waiting-for-triage
Dominant language
Java
Stars
60.2k
Forks
38.8k
Avg merge
5d 2h
Merged PRs (30d)
27

Description

Description:

I noticed that the default thread name prefix used by MVC async execution differs from other task infrastructure components.

Current MVC async executor:

public class RequestMappingHandlerAdapter extends AbstractHandlerMethodAdapter
		implements BeanFactoryAware, InitializingBean {

....
private class MvcSimpleAsyncTaskExecutor extends SimpleAsyncTaskExecutor {
....
      MvcSimpleAsyncTaskExecutor() {
          super("MvcAsync");
      }
....
}

produces thread names like:

MvcAsync1
MvcAsync2

while task scheduling infrastructure commonly uses prefixes such as:

task-1
task-2

using a hyphen separator and lower-case naming.

Is the current MvcAsync prefix intentional for historical or compatibility reasons?

I am not necessarily proposing a direct change because it would alter externally visible thread names and may impact logging or monitoring setups. I wanted to ask whether there has been discussion around standardizing naming conventions across Spring task-related infrastructure, perhaps for future major versions.

Thanks.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by reading RequestMappingHandlerAdapter and its MvcSimpleAsyncTaskExecutor, focusing on the current "MvcAsync" thread-name prefix. Compare it with the task infrastructure naming mentioned in the issue and investigate whether existing compatibility concerns affect standardization. Done means reaching a documented decision or a clearly scoped proposal for any future change.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
backend
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Quiet
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.