elphastori commented on issue #13119: URL: https://github.com/apache/iceberg/issues/13119#issuecomment-2904798383
Thanks for linking the [rest spec](https://github.com/apache/iceberg/blob/91dff9886e6e6c494f6a970129f4c08487c98a0d/open-api/rest-catalog-open-api.yaml#L2018C1-L2041C21). > An opaque token that allows clients to make use of pagination for list APIs > (e.g. ListTables). Clients may initiate the first paginated request by sending an empty > query parameter `pageToken` to the server. Based on this description, the REST catalog is technically behaving as expected. - Works: http://localhost:8181/v1/namespaces/default/tables?pageSize=1&pageToken= - Doesn't work: http://localhost:8181/v1/namespaces/default/tables?pageSize=1 The API expects the first request to have an *empty* `pageToken`, rather than omitting the parameter entirely. While this aligns with the current spec, it’s not the most intuitive developer experience. Ideally, I would suggest updating the spec description to something like: > Clients can initiate the first paginated request *either* without the `pageToken` query parameter or with an empty one. -- 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
