spicedb icon indicating copy to clipboard operation
spicedb copied to clipboard

Pagination support for Expand API

Open josephschorr opened this issue 4 years ago • 8 comments

Add support for pagination into the Expand API

Should ideally make use of a generic pagination system over the dispatcher that can be shared with Lookup

josephschorr avatar Aug 30 '21 19:08 josephschorr

What is the current status of this issue?

It is highly required by our processes. Can we make a contribution to this or is there any ongoing development for this?

mstrYoda avatar Feb 02 '23 15:02 mstrYoda

@mstrYoda No updates and not currently scheduled. We would be happy to accept a contribution.

josephschorr avatar Feb 02 '23 15:02 josephschorr

One thing to note: we should support continuation cursors, not pagination

josephschorr avatar Feb 02 '23 16:02 josephschorr

One thing to note: we should support continuation cursors, not pagination

Thank you for your response, I will investigate on this for a while 👍

mstrYoda avatar Feb 02 '23 21:02 mstrYoda

@mstrYoda Cursor support has landed for LookupResources and ReadRelationships. if you still intend to do this, you can reuse the cursor package.

josephschorr avatar May 23 '23 20:05 josephschorr

@mstrYoda Cursor support has landed for LookupResources and ReadRelationships. if you still intend to do this, you can reuse the cursor package.

Thank you for all of your effort, we are closely following new releases of spicedb the cursor support made us very happy :)

If I wait for a while I guess this will be implemented :D I will take a look at the new developments 👍

mstrYoda avatar May 23 '23 20:05 mstrYoda

@mstrYoda Given that cursors will land soon-ish for LookupSubjects, I suspect we won't add cursors to Expand unless there is a need. Do you still need this support?

josephschorr avatar Jun 23 '23 18:06 josephschorr

@mstrYoda Given that cursors will land soon-ish for LookupSubjects, I suspect we won't add cursors to Expand unless there is a need. Do you still need this support?

Hi @josephschorr, cursor for LookupSubjects is cover most of our requirements. Thank you, I do not see any urgent need for Expand.

mstrYoda avatar Jun 24 '23 09:06 mstrYoda