(drill) branch master updated: DRILL-8524: Drill Adding Duplicate Parameters in Offset Paginator (#2988)

[cgivre]

This is an automated email from the ASF dual-hosted git repository.

cgivre pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/drill.git


The following commit(s) were added to refs/heads/master by this push:
     new 153093778f DRILL-8524: Drill Adding Duplicate Parameters in Offset 
Paginator (#2988)
153093778f is described below

commit 153093778f9d6414e62860a8660cc764e577db93
Author: Charles S. Givre <cgi...@apache.org>
AuthorDate: Thu May 8 09:05:19 2025 -0400

    DRILL-8524: Drill Adding Duplicate Parameters in Offset Paginator (#2988)
---
 contrib/storage-http/pom.xml                                       | 2 +-
 .../apache/drill/exec/store/http/paginator/OffsetPaginator.java    | 7 ++-----
 2 files changed, 3 insertions(+), 6 deletions(-)

diff --git a/contrib/storage-http/pom.xml b/contrib/storage-http/pom.xml
index d49e2ab7d1..9796978992 100644
--- a/contrib/storage-http/pom.xml
+++ b/contrib/storage-http/pom.xml
@@ -31,7 +31,7 @@
   <name>Drill : Contrib : Storage : HTTP</name>
 
   <properties>
-    <okhttp.version>4.9.3</okhttp.version>
+    <okhttp.version>4.12.0</okhttp.version>
   </properties>
 
   <dependencies>
diff --git 
a/contrib/storage-http/src/main/java/org/apache/drill/exec/store/http/paginator/OffsetPaginator.java
 
b/contrib/storage-http/src/main/java/org/apache/drill/exec/store/http/paginator/OffsetPaginator.java
index eed3c2384b..2f30c6d428 100644
--- 
a/contrib/storage-http/src/main/java/org/apache/drill/exec/store/http/paginator/OffsetPaginator.java
+++ 
b/contrib/storage-http/src/main/java/org/apache/drill/exec/store/http/paginator/OffsetPaginator.java
@@ -73,11 +73,8 @@ public class OffsetPaginator extends Paginator {
       throw new NoSuchElementException();
     }
 
-    builder.removeAllEncodedQueryParameters(offsetParam);
-    builder.removeAllEncodedQueryParameters(limitParam);
-
-    builder.addQueryParameter(offsetParam, String.valueOf(offset));
-    builder.addQueryParameter(limitParam, String.valueOf(pageSize));
+    builder.setEncodedQueryParameter(offsetParam, String.valueOf(offset));
+    builder.setEncodedQueryParameter(limitParam, String.valueOf(pageSize));
     offset += pageSize;
 
     return builder.build().url().toString();