Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Optimize get all by collection #485

Merged
merged 2 commits into from
Dec 26, 2021
Merged

Optimize get all by collection #485

merged 2 commits into from
Dec 26, 2021

Conversation

amirylm
Copy link
Contributor

@amirylm amirylm commented Dec 22, 2021

Partially resolve #484

  • Read multiple items w/o allocating slice for results

@amirylm amirylm marked this pull request as ready for review December 23, 2021 07:15
@amirylm amirylm requested a review from nivBlox December 26, 2021 10:45
@amirylm amirylm merged commit 6462fe5 into stage Dec 26, 2021
@amirylm amirylm deleted the get-all-opt branch December 26, 2021 11:42
@nivBlox nivBlox mentioned this pull request Dec 29, 2021
Merged
@amirylm amirylm self-assigned this Feb 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nivBlox nivBlox Awaiting requested review from nivBlox

Assignees

@amirylm amirylm

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@amirylm