jackye1995 commented on PR #9866: URL: https://github.com/apache/iceberg/pull/9866#issuecomment-1979323286
I think we still need to figure out `GET /tables?pageToken` vs `GET /tables?pageToken=` or both. As I said in the old PR, in https://www.youtube.com/watch?v=uAQVGd5zV4I, starting at 45:04, seems like @rdblue said we should support both there. And my understanding is that if we support `pageToken=`, then `pageToken` is automatically supported by default, and both presents value as empty string. It would be awkward to fail one and succeed another. -- 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
