Skip to content

Added configuration for reactive scheduler - #1032

Merged
kdavisk6 merged 2 commits into
OpenFeign:masterfrom
SimY4:master
Aug 7, 2019
Merged

Added configuration for reactive scheduler#1032
kdavisk6 merged 2 commits into
OpenFeign:masterfrom
SimY4:master

Conversation

@SimY4

@SimY4 SimY4 commented Aug 7, 2019

Copy link
Copy Markdown
Contributor

Added configuration for reactive schedulers.

Sometimes it's quite an extreme measure to schedule execution on an unbound thread pool. This PR mitigates that by adding control which scheduler to use for request execution.

@velo velo left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm

@kdavisk6 kdavisk6 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@SimY4

Thanks for this change. This will make our next release.

@kdavisk6 kdavisk6 added this to the 10.3.0 milestone Aug 7, 2019
@kdavisk6
kdavisk6 merged commit 23abb50 into OpenFeign:master Aug 7, 2019
velo pushed a commit that referenced this pull request Oct 7, 2024
* Added configuration for reactive scheduler

* make final things explicitly final.
velo pushed a commit that referenced this pull request Oct 8, 2024
* Added configuration for reactive scheduler

* make final things explicitly final.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants