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
[PoolManager] - Synchronized -> Semaphore, wrap side-effects in Effect[F].delay #1934
Merged
rossabaker
merged 22 commits into
http4s:master
from
tbrown1979:attempt-at-purifying-pool-mgr
Aug 17, 2018
Merged
Changes from 6 commits
Commits
Show all changes
22 commits
Select commit
Hold shift + click to select a range
4b1c31f
Using a Semaphore instead of synchronizing
4c58fc2
Attempt at purifying functions
d5584a0
More purifying of pool manager
a04e026
Purifying the code
2f17e3c
Creation of a PoolManager is an effect at this point
05b3b7c
Merge branch 'master' into attempt-at-purifying-pool-mgr
ba0e8fa
Make sure to acquire permit for disposeConnection
a35ec4e
Make sure to properly releaseNonRecyclable
ed77b3a
Forgo to format
24bc93a
Format blaze-client module as well
6dc5c7c
Removing commented imports
c6ee932
Removing the PooledHttp1Client
2558d10
Switching PoolManager and upstream things to Concurrent instead of Ef…
fc3841a
Just have Http1Client be constrained by ConcurrentEffect
d880407
Start effect after we incrConnection
3773e37
Getting things to compile with global ec
e40f9c3
Update MimeGenerator.scala
tbrown1979 136df85
Use IOApp for client examples
ae0aa92
Merge branch 'attempt-at-purifying-pool-mgr' of https://github.com/tb…
14a39ad
MimeLoaderApp is now an IOApp
0ec71df
Using .withPermit instead of explicit acquire/release
f88439e
Hit callback in the else
File filter
Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If we're going to force
toIO
, might as well make the whole call returnF[Client[F]]
instead. There's no need to maintain source compat on a deprecated method since M7 of 0.18.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jmcardon Is it okay to just remove the
PooledHttp1Client
then? I only did this so that we could keep it around, but figured I'd get questions about it.