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

Added mergeSorted to merge and corresponding Spec #543

Closed
wants to merge 2 commits into
base: series/0.8a
from

Conversation

Projects
None yet
5 participants
@alexwei

alexwei commented Feb 17, 2016

Merging two already sorted streams into one output stream of These, which indicates whether the values is only present on the left, on the right or in both, matched by sorted key calculated by provided functions.

The algorithm is adapted from the merge sorted stream algorithm from https://softwaremill.com/comparing-akka-stream-scalaz-stream/ by @adamw.

@puffnfresh

This comment has been minimized.

Show comment
Hide comment
@puffnfresh

puffnfresh commented Feb 17, 2016

👍

@tonymorris

This comment has been minimized.

Show comment
Hide comment
@tonymorris

tonymorris Feb 18, 2016

¯\_(ツ)_/¯

tonymorris commented Feb 18, 2016

¯\_(ツ)_/¯
@mpilquist

This comment has been minimized.

Show comment
Hide comment
@mpilquist

mpilquist Apr 25, 2016

Member

Perhaps it would be easier/better to add this to a new fs2-scalaz interop project. Then you can use Order and \&/ without changes. If there's interest in such a project, let me know and I'll get it going.

Member

mpilquist commented Apr 25, 2016

Perhaps it would be easier/better to add this to a new fs2-scalaz interop project. Then you can use Order and \&/ without changes. If there's interest in such a project, let me know and I'll get it going.

@mpilquist

This comment has been minimized.

Show comment
Hide comment
@mpilquist

mpilquist Jun 4, 2016

Member

@alexwei We could add this method to https://github.com/functional-streams-for-scala/fs2-scalaz if you are still interested.

Member

mpilquist commented Jun 4, 2016

@alexwei We could add this method to https://github.com/functional-streams-for-scala/fs2-scalaz if you are still interested.

@mpilquist mpilquist closed this Jun 19, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment