-
Notifications
You must be signed in to change notification settings - Fork 76
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
Add PekkoStream #572
Add PekkoStream #572
Conversation
cf9fe50
to
5c24d51
Compare
704c268
to
d02b633
Compare
Thanks for your patient review. @cchantep I addressed remaining comments. |
d02b633
to
ee253a5
Compare
need fixes |
c819d3e
to
0f5e6d2
Compare
I am not able to find why it fails, do you have any hints?
|
0f5e6d2
to
4ff56fa
Compare
Run |
It passed when I run it locally:
🤔 |
|
c51c206
to
03ac20e
Compare
I am not able to reproduce it locally neither with |
353eecb
to
d3b36bb
Compare
d3b36bb
to
3fe3040
Compare
Hello, thanks for the work. Any idea when this will be released? I was upgrading to Play 3 and Pekko but stuck with this for the moment. |
Hello @cchantep any updates on this release? |
Pull Request Checklist
Fixes
Adds
PekkoStream
to be used by Pekko users.Purpose
Many applications are now migrating to Pekko due to Akka's license change. Some of them might be using AkkaStream from anorm, this supports them with their migration.
Background Context
I do not think there is a better approach, we have to duplicate the code.
References
#571