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.

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

OptionalConcreteLimit and OptionalCursor on LookupSubjects doesn't work. #1964

Closed
karlwang1983 opened this issue Jul 2, 2024 · 2 comments
Closed
Labels
kind/bug Something is broken or regressed

Comments

@karlwang1983
Copy link

What platforms are affected?

macos

What architectures are affected?

arm64

What SpiceDB version are you using?

v1.33.0

Steps to Reproduce

1 Create 10 editor relations of user and documents.
2 Invoke LookupSubject API with subject request:
LookupSubjectsRequest { Resource = "document1", SubjectObjectType = "user", Permission = "editor", Consistency = new SpiceDB.Consistency { FullyConsistent = true }, OptionalConcreteLimit = 2, OptionalCursor = null };
3 Expect return 2 subjects.
4 Actually return 10 subjects, after_result_cursor is null.

Expected Result

Expect return 2 subjects

Actual Result

Actually return 10 subjects, after_result_cursor is null.

@karlwang1983 karlwang1983 added the kind/bug Something is broken or regressed label Jul 2, 2024
@vroldanbet
Copy link
Contributor

Hey @karlwang1983, support for cursors in LookupSubjects is not yet merged: #1379

This is an artifact of the API being published, and already merged into spicedb but the actual implementation not being in place.

#1379 is already reviewed, we are conducting more tests, and should be planned for 1.35.

FYI @josephschorr

@karlwang1983
Copy link
Author

@vroldanbet Thank you for your prompt reply. Looking forward to the release of the version 1.35

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Something is broken or regressed
Projects
None yet
Development

No branches or pull requests

2 participants