Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Harmonize ChannelRegistration to use Executor rather than TaskExecutor #32129

Closed
snicoll opened this issue Jan 26, 2024 · 0 comments
Closed

Harmonize ChannelRegistration to use Executor rather than TaskExecutor #32129

snicoll opened this issue Jan 26, 2024 · 0 comments
Assignees
@snicoll
Labels
in: messaging Issues in messaging modules (jms, messaging) type: enhancement A general enhancement
Milestone
6.2.0-M1

Comments

@snicoll
Copy link
Member

snicoll commented Jan 26, 2024

With the fix made in #32081, we noticed that ChannelRegistration requires a TaskExecutor but what it really needs is an Executor really. We'll use this issue to harmonize the types.
@snicoll snicoll added in: messaging Issues in messaging modules (jms, messaging) type: enhancement A general enhancement labels Jan 26, 2024
@snicoll snicoll added this to the 6.2.0-M1 milestone Jan 26, 2024
@snicoll snicoll self-assigned this Jan 26, 2024
snicoll added a commit to snicoll/spring-framework that referenced this issue Jan 26, 2024
@snicoll
Harmonize WebSocket message broker to use Executor
cdca5ac 
This commit harmonizes the configuration of the WebSocket message
broker to use Executor rather than TaskExecutor as only the former
is enforced. This lets custom configuration to use a wider range
of implementations.

Closes spring-projectsgh-32129
snicoll added a commit to snicoll/spring-framework that referenced this issue Jan 31, 2024
@snicoll
Harmonize WebSocket message broker to use Executor
705da29 
This commit harmonizes the configuration of the WebSocket message
broker to use Executor rather than TaskExecutor as only the former
is enforced. This lets custom configuration to use a wider range
of implementations.

Closes spring-projectsgh-32129
izeye added a commit to izeye/spring-framework that referenced this issue Mar 23, 2024
@izeye
Fix Javadoc since to ChannelRegistration.executor()
5286716 
See spring-projectsgh-32129
@izeye izeye mentioned this issue Mar 23, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@snicoll snicoll

Labels
in: messaging Issues in messaging modules (jms, messaging) type: enhancement A general enhancement
Projects
None yet
Milestone
6.2.0-M1
Development

No branches or pull requests
1 participant
@snicoll