This is an automated email from the ASF dual-hosted git repository.
snazy pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/polaris.git
The following commit(s) were added to refs/heads/main by this push:
new b1098a5dd2 Nit: Accept and treat empty page token string as not
present (#3792)
b1098a5dd2 is described below
commit b1098a5dd27108a3d9bf6a455fba9fac5b3dea35
Author: Robert Stupp <[email protected]>
AuthorDate: Wed Feb 18 18:30:01 2026 +0100
Nit: Accept and treat empty page token string as not present (#3792)
This matches the Iceberg/Java implementation
---
.../org/apache/polaris/core/persistence/pagination/PageTokenUtil.java | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git
a/polaris-core/src/main/java/org/apache/polaris/core/persistence/pagination/PageTokenUtil.java
b/polaris-core/src/main/java/org/apache/polaris/core/persistence/pagination/PageTokenUtil.java
index 821909a720..d28a5dca7d 100644
---
a/polaris-core/src/main/java/org/apache/polaris/core/persistence/pagination/PageTokenUtil.java
+++
b/polaris-core/src/main/java/org/apache/polaris/core/persistence/pagination/PageTokenUtil.java
@@ -121,7 +121,9 @@ final class PageTokenUtil {
@Nullable String requestedPageToken,
@Nullable Integer requestedPageSize,
BooleanSupplier shouldDecodeToken) {
- if (requestedPageToken != null && shouldDecodeToken.getAsBoolean()) {
+ if (requestedPageToken != null
+ && !requestedPageToken.isEmpty()
+ && shouldDecodeToken.getAsBoolean()) {
var bytes = Base64.getUrlDecoder().decode(requestedPageToken);
try {
var pageToken = SMILE_MAPPER.readValue(bytes, PageToken.class);