[jira] [Commented] (DRILL-8365) HTTP Plugin Places Parameters in Wrong Place

2022-12-05 Thread ASF GitHub Bot (Jira)


[ 
https://issues.apache.org/jira/browse/DRILL-8365?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17643319#comment-17643319
 ] 

ASF GitHub Bot commented on DRILL-8365:
---

cgivre merged PR #2715:
URL: https://github.com/apache/drill/pull/2715




> HTTP Plugin Places Parameters in Wrong Place
> 
>
> Key: DRILL-8365
> URL: https://issues.apache.org/jira/browse/DRILL-8365
> Project: Apache Drill
>  Issue Type: Bug
>  Components: Storage - HTTP
>Affects Versions: 1.20.2
>Reporter: Charles Givre
>Assignee: Charles Givre
>Priority: Major
> Fix For: 1.20.3
>
>
> When the requireTail option is set to true, and pagination is enabled, the 
> HTTP plugin puts the required parameters in the wrong place in the URL.  This 
> PR fixes that.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Commented] (DRILL-8365) HTTP Plugin Places Parameters in Wrong Place

2022-12-04 Thread ASF GitHub Bot (Jira)


[ 
https://issues.apache.org/jira/browse/DRILL-8365?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17643074#comment-17643074
 ] 

ASF GitHub Bot commented on DRILL-8365:
---

cgivre opened a new pull request, #2715:
URL: https://github.com/apache/drill/pull/2715

   # [DRILL-8365](https://issues.apache.org/jira/browse/DRILL-8365): HTTP 
Plugin Places Parameters in Wrong Place
   
   ## Description
   This PR fixes a bug when a user configures a HTTP plugin with the 
`requireTail` option.  The plugin was putting the URL parameters in the wrong 
place when pagination is used.  This fixes that.
   
   ## Documentation
   N/A
   
   ## Testing
   Added unit test and ran existing unit tests.




> HTTP Plugin Places Parameters in Wrong Place
> 
>
> Key: DRILL-8365
> URL: https://issues.apache.org/jira/browse/DRILL-8365
> Project: Apache Drill
>  Issue Type: Bug
>  Components: Storage - HTTP
>Affects Versions: 1.20.2
>Reporter: Charles Givre
>Assignee: Charles Givre
>Priority: Major
> Fix For: 1.20.3
>
>
> When the requireTail option is set to true, and pagination is enabled, the 
> HTTP plugin puts the required parameters in the wrong place in the URL.  This 
> PR fixes that.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)