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

v0 Read API does not scale to tens of thousands of results #3

Closed
jzelinskie opened this issue Aug 16, 2021 · 2 comments
Closed

v0 Read API does not scale to tens of thousands of results #3

jzelinskie opened this issue Aug 16, 2021 · 2 comments
Assignees
@jakedt
Labels
area/api v0 Affects the v0 API area/perf Affects performance or scalability

Comments

@jzelinskie
Copy link
Member

authzed.api.v0.ACLService.Read() cannot support reading queries with large result sets.

This will have to be added as some form of pagination in order to remain backwards compatible.

A future API could use gRPC streams instead of pagination.
@jzelinskie jzelinskie added area/api v0 Affects the v0 API area/perf Affects performance or scalability labels Aug 16, 2021
@josephschorr
Copy link
Member

Read V1 is now streaming: #74

@jzelinskie jzelinskie changed the title Read API does not scale to tens of thousands of results v0 Read API does not scale to tens of thousands of results Sep 30, 2021
@jzelinskie
Copy link
Member Author

Closing this since it was specific to v0.

github-merge-queue bot pushed a commit that referenced this issue Jul 25, 2023
@josephschorr
Merge pull request #3 from ecordell/cel-best-practices
afaf313 
bump all dependencies
@ecordell ecordell mentioned this issue Dec 18, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jakedt jakedt

Labels
area/api v0 Affects the v0 API area/perf Affects performance or scalability
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@jzelinskie @jakedt @josephschorr