Skip to content
This repository has been archived by the owner on May 16, 2023. It is now read-only.

Remove ForkJoinPool #489

Closed
wants to merge 2 commits into from
Closed

Remove ForkJoinPool #489

wants to merge 2 commits into from

Conversation

michael-burwig
Copy link
Member

@michael-burwig michael-burwig commented Jun 9, 2020

  • Removes ForkJoinPool and processes real requests in their original thread.

Open addressable questions:

  • Should the fake request delay ever be 0?

@michael-burwig michael-burwig changed the title Replace ForkJoinPool with ScheduledThreadPool Remove ForkJoinPool Jun 9, 2020
@michael-burwig michael-burwig marked this pull request as draft June 9, 2020 09:50
@sonarqubecloud
Copy link

sonarqubecloud bot commented Jun 9, 2020

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@michael-burwig michael-burwig marked this pull request as ready for review June 9, 2020 11:16
@nils4cosee
Copy link
Contributor

nils4cosee commented Jun 9, 2020

I just noticed this PR. I have submitted a similar PR in #495 , to resolve #399

@stevesap stevesap closed this Jun 9, 2020
@michael-burwig michael-burwig deleted the fix/remove-fork-join branch June 10, 2020 20:09
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants