jackye1995 commented on code in PR #9660:
URL: https://github.com/apache/iceberg/pull/9660#discussion_r1494891220
##########
open-api/rest-catalog-open-api.yaml:
##########
@@ -1581,6 +1616,12 @@ components:
type: string
example: [ "accounting", "tax" ]
+ NextPageToken:
+ description:
+ An opaque next page token, when non-null and non-empty this indicates
that more results can be returned by server.
Review Comment:
Actually, why do we have a definition of page token and next page token? I
think this should just be `PageToken`, and the page-token query parameter
schema should point to this, so does the `next-page-token` fields in the
response. What do you think?
--
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
