[
https://issues.apache.org/jira/browse/DRILL-8524?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17949615#comment-17949615
]
ASF GitHub Bot commented on DRILL-8524:
---------------------------------------
cgivre opened a new pull request, #2988:
URL: https://github.com/apache/drill/pull/2988
# [DRILL-8524](https://issues.apache.org/jira/browse/DRILL-XXXX): Drill
Adding Duplicate Parameters in Offset Paginator
## Description
This PR fixes a minor bug with the Offset paginator. When the URL params
have a `$` or other control character, the pagination was not properly updating
the URL and adding duplicate parameters to the URL. This PR solves that.
Additionally, this PR bumps OkHTTP to the latest version.
## Documentation
No user facing changes.
## Testing
Tested against user provided API. Ran existing unit tests.
> Drill Adding Duplicate Parameters in Offset Paginator
> -----------------------------------------------------
>
> Key: DRILL-8524
> URL: https://issues.apache.org/jira/browse/DRILL-8524
> Project: Apache Drill
> Issue Type: Bug
> Components: Storage - HTTP
> Affects Versions: 1.21.2
> Reporter: Charles Givre
> Assignee: Charles Givre
> Priority: Major
> Fix For: 1.22.0
>
>
> When parameters contain control characters, Drill was not properly forming
> the URLs. This PR fixes that.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
