jackye1995 commented on code in PR #9660:
URL: https://github.com/apache/iceberg/pull/9660#discussion_r1511679559
##########
open-api/rest-catalog-open-api.yaml:
##########
@@ -1581,6 +1608,17 @@ components:
type: string
example: [ "accounting", "tax" ]
+ PageToken:
+ description:
+ An opaque token which allows clients to make use of pagination for a
list API (e.g. ListTables).
+ Clients will initiate the first paginated request by sending an empty
`pageToken` e.g. `GET /tables?pageToken` or `GET /tables?pageToken=`
Review Comment:
Trying to remember what was decided during the community sync discussion,
https://www.youtube.com/watch?v=uAQVGd5zV4I, starting at 45:04, seems like you
said we should support both there. But maybe it's a misunderstanding of what
Rahil meant there when he was talking about the first option of value not
present.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org
For additional commands, e-mail: issues-h...@iceberg.apache.org
