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

Upgrade to Reactor Dysprosium GA #23665

Closed
bclozel opened this issue Sep 18, 2019 · 1 comment
Closed

Upgrade to Reactor Dysprosium GA #23665

bclozel opened this issue Sep 18, 2019 · 1 comment
Labels
status: duplicate A duplicate of another issue type: dependency-upgrade A dependency upgrade

Comments

@bclozel
Copy link
Member

bclozel commented Sep 18, 2019

We need to upgrade to the GA version once it's released.
@bclozel bclozel added the type: dependency-upgrade A dependency upgrade label Sep 18, 2019
@bclozel bclozel added this to the 5.2 GA milestone Sep 18, 2019
bclozel added a commit that referenced this issue Sep 18, 2019
@bclozel
Use Reactor's new Schedulers.boundedElastic()
74897bc 
Prior to this commit, Spring Framework would use `Schedulers.elastic()`
in places where we needed to process blocking tasks in a reactive
environment.

With reactor/reactor-core#1804, a new `Schedulers.boundedElastic()`
scheduler is available and achieves the same goal with added security;
it guarantees that resources are bounded.

This commit uses that new scheduler in the standard websocket client,
since the underlying API is blocking for the connection phase and we
need to schedule that off a web server thread.

Closes gh-23661
See gh-23665
@jhoeller jhoeller changed the title Upgrade to Reactor Dysprosium Upgrade to Reactor Dysprosium GA Sep 23, 2019
@snicoll
Copy link
Member

snicoll commented Sep 26, 2019

Duplicate of #23695 (sorry I didn't find it when I created the duplicate).

@snicoll snicoll closed this as completed Sep 26, 2019
@snicoll snicoll removed this from the 5.2 GA milestone Sep 26, 2019
@snicoll snicoll added the status: duplicate A duplicate of another issue label Sep 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
status: duplicate A duplicate of another issue type: dependency-upgrade A dependency upgrade
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@bclozel @snicoll