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
refactor(snapshots): Replace Stream.toList
and the for each cycle to Stream.forEachOrdered
#12576
refactor(snapshots): Replace Stream.toList
and the for each cycle to Stream.forEachOrdered
#12576
Conversation
…o `Stream.forEachOrdered`
Hello, reviewer! |
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.
🚀 Thanks @aivinog1 for your contribution.
Since this is reasonable change irrespective of performance improvement, I'm merging this without running a benchmark. You would see a difference only with large number of snapshot files anyway.
bors r+
12576: refactor(snapshots): Replace `Stream.toList` and the for each cycle to `Stream.forEachOrdered` r=deepthidevaki a=aivinog1 ## Description <!-- Please explain the changes you made here. --> I replaced the collection with a list and the for each cycle to the `Stream.forEactOrdered` call. It should reduce memory footprint and possibly improve latency, especially on a large number of snapshot files. ## Related issues <!-- Which issues are closed by this PR or are related --> closes #12575 Co-authored-by: Alexey Vinogradov <vinogradov.a.i.93@gmail.com>
Build failed: |
Smoke test failed. bors retry |
Build succeeded: |
Thanks @deepthidevaki 🎉 |
@aivinog1 backport workflow is running. Please assign it to me when the backport PRs are created. |
@deepthidevaki It failed (I think because I drop my branch, but I just restored it now), could you restart the workflow, please? 🙂 Or check out what went wrong. |
Let's try again. /backport |
Backport failed for Please cherry-pick the changes locally. git fetch origin stable/8.0
git worktree add -d .worktree/backport-12576-to-stable/8.0 origin/stable/8.0
cd .worktree/backport-12576-to-stable/8.0
git checkout -b backport-12576-to-stable/8.0
ancref=$(git merge-base 02d1df8ac1b6f5484f1e9d4c1f19c8a5712176b1 499036719908a9487853a727c54e04c637119498)
git cherry-pick -x $ancref..499036719908a9487853a727c54e04c637119498 |
Successfully created backport PR for |
Successfully created backport PR for |
12662: [Backport stable/8.1] fix: do not retake backup if it already exists r=deepthidevaki a=deepthidevaki Backport of #12626 to stable/8.1. relates to #12623 12665: [Backport stable/8.1] refactor(snapshots): Replace `Stream.toList` and the for each cycle to `Stream.forEachOrdered` r=deepthidevaki a=backport-action # Description Backport of #12576 to `stable/8.1`. relates to #12575 Co-authored-by: Deepthi Devaki Akkoorath <deepthidevaki@gmail.com> Co-authored-by: Deepthi Devaki Akkoorath <deepthidevaki@users.noreply.github.com> Co-authored-by: Alexey Vinogradov <vinogradov.a.i.93@gmail.com>
Description
I replaced the collection with a list and the for each cycle to the
Stream.forEactOrdered
call. It should reduce memory footprint and possibly improve latency, especially on a large number of snapshot files.Related issues
closes #12575
Definition of Done
Not all items need to be done depending on the issue and the pull request.
Code changes:
backport stable/1.3
) to the PR, in case that fails you need to create backports manually.Testing:
Documentation:
Other teams:
If the change impacts another team an issue has been created for this team, explaining what they need to do to support this change.
Please refer to our review guidelines.