eric-maynard commented on PR #1555: URL: https://github.com/apache/polaris/pull/1555#issuecomment-2902728893
@dimas-b per the spec, the request provides `page-token` and `page-size`. So there is no `A1`, but the presence of `A2` or `A3` implies `A1`. Turning this `page-token` into a `PageToken` actually used to be done with persistence-specific logic, but I refactored that out in response to [your comment here](https://github.com/apache/polaris/pull/1528#discussion_r2078313029) which indeed simplified the code significantly. I'll make some changes to put some of this logic back -- but since the page token types are all shared, I'll try to keep the actual types in core for now. -- 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...@polaris.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
