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

Rename spring-web-reactive to spring-webflux [SPR-15190] #19756

Closed
spring-issuemaster opened this Issue Jan 26, 2017 · 1 comment

Comments

Projects
None yet
2 participants
@spring-issuemaster
Copy link
Collaborator

spring-issuemaster commented Jan 26, 2017

Rossen Stoyanchev opened SPR-15190 and commented

The current module name spring-web-reactive is a bit of a misnomer because it does not contain all "reactive, web" functionality as its name advertises. The low level "reactive, web" infrastructure for HTTP runtimes and Reactive Streams is in spring-web. The higher level, generic web framework support comparable to the Servlet API is also in spring-web.

The situation mirrors the spring-webmvc module which contains the Spring MVC framework but also builds on a lot of infrastructure in the general spring-web module. We need a similar, distinct name for the reactive web endpoint framework.

Note also that the module name change does not have to impact the package structure. In other words just like Spring MVC actually lives in org.springframework.web.servlet while its module name is spring-webmvc, the top level package in the spring-web-reactive module can remain org.springframework.web.reactive while the module name is changed.


Affects: 5.0 M4

Referenced from: commits 7399956, 080a3d6, fafd2d2

@spring-issuemaster

This comment has been minimized.

Copy link
Collaborator Author

spring-issuemaster commented Jan 26, 2017

Rossen Stoyanchev commented

Additional tasks:

  • Update documentation reference chapter
  • Update Javadoc (e.g. package-info or main classes)
  • Replace a few occurrences of "Spring MVC" in the Javadoc
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment