Release for 2.0 #333
Open
Release for 2.0 #333
Conversation
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 was referenced Oct 3, 2024
Closed
Closed
nomisRev
approved these changes
Oct 10, 2024
content/blog/2024-10-04-arrow-2-0.md
Outdated
Comment on lines
32
to
34
| between fail-first and [accumulation of errors](/learn/typed-errors/working-with-typed-errors/#accumulating-different-computations). Until now, the latter mode | ||
| required using `parZip` and `parMap`, which sometimes obscure the actual | ||
| flow of the computation. |
There was a problem hiding this comment.
parZip and parMap are used for parallel operations. Did you mean zipOrAccumulate here?
content/blog/2024-10-04-arrow-2-0.md
Outdated
Comment on lines
43
to
46
| parZip( | ||
| { checkOneThing() }, | ||
| { checkOtherThing() } | ||
| ) { a, b -> doSomething(a, b) } |
There was a problem hiding this comment.
Suggested change
| parZip( | |
| { checkOneThing() }, | |
| { checkOtherThing() } | |
| ) { a, b -> doSomething(a, b) } | |
| zipOrAccumulate( | |
| { checkOneThing() }, | |
| { checkOtherThing() } | |
| ) { a, b -> doSomething(a, b) } |
These examples are using context receivers, and/or omitting quite some details. Is it important for us the examples are real-life, and copy-paste able?
| usual `async` mechanism, ensuring that all `Deferred` values are `await`ed | ||
| once the first one is requested. That means that the previous code behaves | ||
| identically to the following, that is, the call `file.await()` implicitly | ||
| awaits every `async` defined up to that point. |
There was a problem hiding this comment.
Do we need to add the details why this is important for Raise, or how not using this could result in hanging?
There was a problem hiding this comment.
Do you have a 1-sentence summary of this I could use?
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
This PR puts together a bunch of other PRs, to check coherence between them:
Fixes #308