Skip to content

Support for the Servlet Async API #1449

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.

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

Open
mdeinum opened this issue Jan 16, 2025 · 2 comments
Open

Support for the Servlet Async API #1449

mdeinum opened this issue Jan 16, 2025 · 2 comments
Labels
type: enhancement A general enhancement

Comments

@mdeinum
Copy link
Contributor

mdeinum commented Jan 16, 2025

Currently Spring WS only leverages the regular Servlet API with blocking the request handling thread. This could be improved by also allowing support for the async features of the Servlet API. Maybe parts of the async support in Spring MVC can be leveraged for this.

@snicoll snicoll added the status: waiting-for-triage An issue we've not yet triaged label Feb 19, 2025
@snicoll snicoll added type: enhancement A general enhancement and removed status: waiting-for-triage An issue we've not yet triaged labels Apr 4, 2025
@snicoll snicoll added this to the General Backlog milestone Apr 4, 2025
@snicoll
Copy link
Member

snicoll commented Apr 4, 2025

I don't know how involved that would be but I'd rather move Spring-WS higher in the stack and make it a better citizen of Spring MVC. @rstoyanchev mentioned that this should be doable so we need to find quality time to investigate what to do.

@mdeinum
Copy link
Contributor Author

mdeinum commented Apr 8, 2025

It would probably involve copying/reusing some parts of the Async processing that is available in the DispatcherServlet.

There is a reason that the MessageDispatcherServlet is the way it is (or that it even exists). I don't recall the exact reason but I do remember that @poutsma did a presentation on this years ago (probably during the inception of Spring WS). So maybe he has some recollection and tips for this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: enhancement A general enhancement
Projects
None yet
Development

No branches or pull requests

2 participants