Add Parallel traverseFilter functions#3467
Conversation
Codecov Report
@@ Coverage Diff @@
## master #3467 +/- ##
==========================================
- Coverage 91.59% 91.37% -0.23%
==========================================
Files 382 383 +1
Lines 8319 8776 +457
Branches 220 216 -4
==========================================
+ Hits 7620 8019 +399
- Misses 699 757 +58 |
|
|
||
| } | ||
|
|
||
| final class ParallelTraverseFilterOps[T[_], A](private val ta: T[A]) extends AnyVal { |
There was a problem hiding this comment.
I think it'd be better to consolidate these, so that all enrichment methods for T[A] are on a single enrichment class, mostly in order to reduce the number of implicits in cats.syntax.all, but also just to simplify this code. In this case that would mean putting these methods on something named ParallelTraversableOps or ParallelFoldMapAOps, though, which also seems kind of bad.
I think for now this version is probably best, but we should definitely revisit this in Cats 3.
|
this PR brought to you by https://www.youtube.com/watch?v=SMMgwfhPCzM&feature=youtu.be 🙂 |
|
You are fast @LukaJCB Thanks for the addition :) |
| trait AllSyntaxBinCompat5 extends ParallelBitraverseSyntax | ||
|
|
||
| trait AllSyntaxBinCompat6 extends ParallelUnorderedTraverseSyntax | ||
| trait AllSyntaxBinCompat6 extends ParallelUnorderedTraverseSyntax with ParallelTraverseFilterSyntax |
There was a problem hiding this comment.
Forgive me for being out for a while, but just curious why we still extend a binCompat trait instead of the AllSyntax itself?
There was a problem hiding this comment.
Ah, good catch. I agree that this should go directly on AllSyntax, as in #3392.
| trait AllSyntaxBinCompat5 extends ParallelBitraverseSyntax | ||
|
|
||
| trait AllSyntaxBinCompat6 extends ParallelUnorderedTraverseSyntax | ||
| trait AllSyntaxBinCompat6 extends ParallelUnorderedTraverseSyntax with ParallelTraverseFilterSyntax |
There was a problem hiding this comment.
Ah, good catch. I agree that this should go directly on AllSyntax, as in #3392.
|
👍 on green, but I think we still need one more review. |
|
Thanks @barambani 😊 merging this |
7 participants
Should fix #3465